summaryrefslogtreecommitdiff
path: root/flower
Commit message (Collapse)AuthorAgeFilesLines
* Issue 4961/7: Remove complex{conjugate,divide,exp}, Offset::argDavid Kastrup2016-09-052-36/+0
| | | | They are promoting aspects of Offset not well-suited to graphics.
* Issue 4961/2: Add offset_directed (Real)David Kastrup2016-09-052-0/+37
| | | | This converts an angle in degrees into a unit vector.
* Issue 4961/1: Make Offset::angle_degrees () deliver nicer anglesDavid Kastrup2016-09-051-1/+57
| | | | Multiples of 45 degrees should be returned exactly.
* Issue 4577: Remove the nasty workaround for platforms that don't provideDan Eble2015-09-111-49/+0
| | | | | | | | | std::vector::data(). Replace most code that reached into into Grob_array and modified its internal vector with calls to new member functions such as filter(). Move the definitions of some trivial methods into grob-array.hh.
* Revert "Issue 4550 (1/2) Avoid "using namespace std;" in included files"David Kastrup2015-08-2318-34/+1
| | | | This reverts commit f127e14af04f474d20406ca0e0f76f05061ee103.
* Revert "Issue 4550 (2/2) Avoid "using namespace std;" in included files"David Kastrup2015-08-2342-147/+172
| | | | This reverts commit 59a6d1a06432fc0ca88c3023c646182f389ec1b5.
* Issue 4550 (2/2) Avoid "using namespace std;" in included filesDan Eble2015-08-2042-172/+147
| | | | | | | | | These changes are produced by a rather long shell script that is posted in the code review for this issue. Summary: * remove "using namespace std;" everywhere * add "std::" in *.hh and other included files * add "std::" to functions and lesser-used types in *.cc files
* Issue 4550 (1/2) Avoid "using namespace std;" in included filesDan Eble2015-08-2018-1/+34
| | | | | | | | | These are manual changes in preparation for an automated removal of "using namespace std;". Mostly these are additions of using-declarations for commonly used types and containers (e.g. std::string, std::vector) to *.cc files so that they will continue to build after the big removal.
* Issue 4560: group #include directives at top of fileDan Eble2015-08-203-3/+4
| | | | | This is so that using-declarations can later be added without appearing before #includes.
* Implement inline Real cross_product (Offset, Offset)David Kastrup2015-08-201-0/+8
|
* Issue 4549: Remove dead "!STD_VECTOR" code in flower testsDan Eble2015-08-172-31/+0
|
* Issue 4548: eliminate flip(Direction*)Dan Eble2015-08-171-21/+2
| | | | | Use for (DOWN_and_UP (d)) {...} instead. Also replace the few uses of other_dir() with unary minus.
* Issue 4547: remove unnecessary min() and max() functionsDan Eble2015-08-141-4/+0
|
* Issue 4546: don't redefine comparison operators for std::stringDan Eble2015-08-143-11/+1
|
* Robustify Offset::direction and Offset::lengthDavid Kastrup2015-07-051-2/+12
|
* Issue 4464/2: include hygiene: break cycles, remove duplicate includes etc.David Kastrup2015-07-024-4/+2
|
* Issue 2787: Sanitize usage of -DDEBUG, -DNDEBUG and assertDavid Kastrup2015-05-193-3/+3
| | | | | | | | | | | | | | | | The compiler option -DNDEBUG is no longer being used: -DNDEBUG disables the assert function, and assert is essentially stating that the program cannot useful continue if the assertion is not met. -DNDEBUG is basically an option for compiling an application to a limited amount of ROM when aborting with a diagnostic is not preferable to crashing. This is not the case for LilyPond. So expensive debugging options now are enabled with -DDEBUG instead. There is a new configure option --enable-checking defaulting to "off" for this now. At the current point of time, setting --disable-optimising also has the effect of enabling the checks: this will be retained until Patchy has been adapted to using --enable-checking.
* Remove cygwin_conv_to_posix_pathMasamichi Hosoda2015-04-241-25/+0
| | | | | | | | | | | | Issue 4346 Remove cygwin_conv_to_posix_path cygwin_conv_to_posix_path is deprecated on cygwin (32bit) and not available on cygwin64. It is not needed in a pure cygwin environment.
* compare LilyPond Rationals without unnecessary overflow; issue 4180Keith OHara2015-01-101-3/+3
| | | | | This reverses part of commit 19125bf1, which said that (r-s).sign() was too expensive, but I saw no change (within 0.1%) in compilation times.
* Run grand replace for 2015.Werner Lemberg2015-01-0442-42/+42
|
* Don't verify result of String_convert::dec2intDavid Kastrup2014-09-081-2/+0
| | | | The error behavior is not useful and triggers on things like "00".
* Issue 3925: lilypond -d leads to weird outputDavid Kastrup2014-06-051-2/+2
| | | | | | | | | | | | | | | | | | C++ string manipulation is a fragile bolt-on onto the type system. "-" + 'd' evaluates to a pointer 100 bytes after a constant string "-" in the data section. This fixes this bug, and also changes the fixed message from lilypond: option `-d, `--define-default'' requires an argument to lilypond: option `-d, --define-default' requires an argument In general, it does not appear to make sense quoting the long options while not quoting the short options.
* Polynomial.hh - Make LilyPond compile w/ musl libcFelix Janda2014-02-201-0/+1
| | | | Provided by Felix Janda
* Run grand-replace (issue 3765)Carl Sorensen2014-01-1142-42/+42
| | | | Run make grand-replace to update all coopyright statements
* Issue 3656/2: disambiguate our own ::to_string from std::to_stringDavid Kastrup2013-11-201-12/+12
|
* Issue 3656: Problems building Lilypond 2.17.95 with libc++ for use with llvmDavid Kastrup2013-11-201-0/+1
| | | | std-vector.hh did not include config.hh but relied on it
* Issue 3531: replaced function argument 'string' by 'const string&' where it ↵Frédéric Bron2013-09-1013-60/+60
| | | | | | | | | | | | | | makes sense to avoid unnecessary copying of 'string' objects. Measurements on x86_64 (i7-2760QM, 2.40GHz) Fedora 19 with g++ 4.8.1, with configure --enable-optimising --disable-debugging; tests run 10 times, average elapsed time compared (/usr/bin/time) * Bach, Concerto in E major or violin and strings, BWV 1042 (Mutopia source), 38 pages: $ lilypond score.ly -> master: 15.4s, with patch: -0.1% * lilypond regression tests (1153 .ly files): $ lilypond *.ly -> master: 276.6s, with patch: -2.5%
* Issue 3513: removed unused code: functions that are declared but never ↵Frédéric Bron2013-09-031-5/+0
| | | | defined and stream.hh
* Issue 2758. ly_module_lookup caused deprecation warnings with Guile V2.06.Ian Hulin2012-09-221-0/+8
| | | | | | | | | | | | The V2.17.0 definition of ly_module_lookup in module-scheme.cc uses two functions, scm_sym2var and scm_module_lookup_closure, which issue deprecation warnings in Guile V2.06. A call to the new Guile API function, scm_module_variable, provides the functionality for both deprecated routines, but has not been back-ported to Guile V1.8. This patch adds a conditionally-compiled block when running with a Guile version < V2.0, so that Guile V1 will not use the deprecated scm_ calls in ly_module_lookup, and Guile V2 will use API call scm_module_variable.
* remove top-level const's from declarationsBenkő Pál2012-09-111-1/+1
| | | | | | results of the following searches were checked manually: grep -rE -e 'const( +[a-zA-Z0-9_]*)? *, *$' . grep -rE -e 'const( +[a-zA-Z0-9_]*)?( *,[^;]+)?\) *(const|= *0)? *;' .
* Improvements in vertical skyline approximations (issue 2148).Mike Solomon2012-08-274-51/+213
| | | | | | | | | | | | | | | | | | | | | | | | | The file stencil-integral.cc provides a suite of functions that traverse a stencil and do linear approximations of its components. These are then turned into boxes that are passed to the Skyline constructor. This approximation is used for several vertical skylines including those of VerticalAxisGroup and System. As a result of these more accurate approximations, vertical spacing is more snug between grobs. Additionally, in axis-group-interface.cc, skylines of grobs are no longer compared to a monolithic axis-group skyline but rather all of the component skylines of the axis-group, allowing grobs to be fit under other ones if there is space instead of always shifted over. Two new python scripts allow to visualize the position of skylines. All other changes provide functions that allow for better debugging of Skylines, better approximations of grobs via skylines, and changes to the measurement of distance between grobs via the new Skyline API. This results in a significant time increase in score compilation for objects with complex skylines such as all text grobs. For orchestral scores, the increase is not as steep.
* Run fixcc + astyle2.02.1Graham Percival2012-08-271-2/+2
|
* Make distributed tarball from Git file listJohn Mandereau2012-08-152-13/+0
| | | | | | | | | | | | | | | | | | This removes the requirement of having a GNUmakefile in each and every directory of the source tree; this commit also deletes GNUmakefiles made unnecessary this way. dist-toplevel-txt-files toplevel target is also removed, because it is unneeded outside of dist target and it prevents linking of toplevel generated txt docs to distdir when these docs are already generated. In addition, on suggestions from Han-Wen, if dist is made with a source directory tracked by Git, then * the source directory is required to have no uncommitted changes nor non-ignored untracked files, * the time stamp of all files in the tarball is set to the time stamp of the head of the checked-out branch.
* Run fixcc.py with astyle 2.02.Graham Percival2012-08-102-2/+0
|
* Issue 2704: Add missing "#include <unistd.h>" for use with g++ 4.7Frédéric Bron2012-08-012-0/+4
|
* Issue 2491: Macro for(UP_and_DOWN) and 3 similar.Łukasz Czerwiński2012-04-301-1/+11
| | | | | | Replaces do{ ... } while(flip (&d) != DOWN/UP/... with a macro for(DOWN_and_UP/UP_and_DOWN/....) { } Signed-off-by: David Kastrup <dak@gnu.org>
* Run grand-replace for 2012Graham Percival2012-01-0942-42/+42
|
* Remove unnecessary stdGraham Percival2012-01-091-1/+0
|
* Avoid implicit sign conversion: size_t vs. ssize_tGraham Percival2012-01-091-3/+3
|
* Avoid implicit sign change by doing it explicitlyGraham Percival2012-01-091-3/+3
|
* Avoid deprecated access declarationsGraham Percival2012-01-091-1/+1
| | | | | | | "According the ANSI/ISO Standard, the use of access declarations is considered deprecated. Instead, you should use a using declaration for that purpose" http://www.devx.com/tips/Tip/5707
* Avoid redefining struct String_convert to class.Graham Percival2011-12-141-2/+2
|
* Avoid implicit sign conversionGraham Percival2011-12-141-3/+3
|
* Remove default comparison for binary searchGraham Percival2011-12-141-1/+1
| | | | | | | | | | | | | | | | | | | | clang++ complains about this, and we never rely on the default comparison in our code. /home/gperciva/src/lilypond/flower/include/std-vector.hh:197:36: error: 'T' does not refer to a value Compare less = less<T> (), ^ /home/gperciva/src/lilypond/flower/include/std-vector.hh:193:19: note: declared here template<typename T, typename Compare> ^ /home/gperciva/src/lilypond/flower/include/std-vector.hh:197:40: error: expected expression Compare less = less<T> (),
* Avoid conversion changing signednessGraham Percival2011-12-141-2/+2
| | | | | | | | /home/gperciva/src/lilypond/flower/file-cookie.cc:45:50: error: implicit conversion changes signedness: 'int' to 'size_t' (aka 'unsigned long') [-Werror,-Wsign-conversion] return Memory_out_stream::writer (file, buf, i); ~~~~~~~~~~~~~~~~~ ^
* Run astyle 2.02.Graham Percival2011-12-123-22/+20
|
* Issue 2036: keep make test in flower from warningsDavid Kastrup2011-12-072-3/+6
|
* Fix almost all warnings in flower/ and many in lily/.Reinhold Kainhofer2011-12-0511-37/+46
| | | | | | | Most of the remaining warnings are about vsize -> int casting, where I'll have to find a proper solution (we don't want to cast all appearances of vsize to int explicitly, but rather use vsize wherever possible).
* Warnings: Move all warning-as-error handling to warn.ccReinhold Kainhofer2011-11-142-10/+17
| | | | This finally makes that option apply to ALL warnings, and it considerably cleans up the error/warning functions.
* Don't let yaffut try to demangle when autoconf figures this won't workDavid Kastrup2011-11-111-0/+9
|