| Commit message (Expand) | Author | Age | Files | Lines |
* | Run grand-replace for 2012 | Graham Percival | 2012-01-09 | 42 | -42/+42 |
* | Remove unnecessary std | Graham Percival | 2012-01-09 | 1 | -1/+0 |
* | Avoid implicit sign conversion: size_t vs. ssize_t | Graham Percival | 2012-01-09 | 1 | -3/+3 |
* | Avoid implicit sign change by doing it explicitly | Graham Percival | 2012-01-09 | 1 | -3/+3 |
* | Avoid deprecated access declarations..."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
| Graham Percival | 2012-01-09 | 1 | -1/+1 |
* | Avoid redefining struct String_convert to class. | Graham Percival | 2011-12-14 | 1 | -2/+2 |
* | Avoid implicit sign conversion | Graham Percival | 2011-12-14 | 1 | -3/+3 |
* | Remove default comparison for binary search...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> (),
| Graham Percival | 2011-12-14 | 1 | -1/+1 |
* | Avoid conversion changing signedness.../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);
~~~~~~~~~~~~~~~~~ ^
| Graham Percival | 2011-12-14 | 1 | -2/+2 |
* | Run astyle 2.02. | Graham Percival | 2011-12-12 | 3 | -22/+20 |
* | Issue 2036: keep make test in flower from warnings | David Kastrup | 2011-12-07 | 2 | -3/+6 |
* | Fix almost all warnings in flower/ and many in lily/....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).
| Reinhold Kainhofer | 2011-12-05 | 11 | -37/+46 |
* | Warnings: Move all warning-as-error handling to warn.cc...This finally makes that option apply to ALL warnings, and it considerably cleans up the error/warning functions.
| Reinhold Kainhofer | 2011-11-14 | 2 | -10/+17 |
* | Don't let yaffut try to demangle when autoconf figures this won't work | David Kastrup | 2011-11-11 | 1 | -0/+9 |
* | Correct forward declarations (struct vs. class) | Graham Percival | 2011-11-09 | 1 | -3/+3 |
* | Add vector constructor with size argument | Carl Sorensen | 2011-10-29 | 1 | -0/+4 |
* | Fix 1477: Add (ly:expect-warning msg args) to suppress expected warnings...If a file contains (ly:expect-warning ....), the corresponding warning
string will be added to a list of expected warnings. If the corresponding
warning (or erro) is triggered, it will not be printed to stderr, but the
string will be removed from the list. So, each ly:expect-warning, suppresses
exactly one occurrence of the warning. To suppress a warning multiple times,
call ly:expect-warning multiple times.
After one file is processed, the list of expected warnings is checked.
If it is not empty, it means that an expected warning was not triggered,
which might be a bug. So we print out a warning message about this fact.
This allows the regtests to check proper warning messages, without
polluting the console output with those warning messages.
All warnings that are actually printed in the regtests are a bug.
For translated error message, there are two approaches: If the warning
is created from scheme, simply use
(ly:expect-warning (_ "msg with ~a") "args")
If the message is triggered from C++, the translated string is in printf
syntax, so we need to translate that and then convert it into a format
string for Scheme. This can be done with the new function
ly:translate-cpp-warning-scheme instead of _:
(ly:expect-warning (ly:translate-cpp-warning-scheme "msg with %s") "args")
This patch does not yet adapt all regtests that are supposed to print warnings
to this new approach. This will follow in a separate patch.
| Reinhold Kainhofer | 2011-09-29 | 2 | -4/+67 |
* | Loglevels: Add basic_progress and replace success...-) Add basic_progress function
-) use it for 'Processing xxx.ly...' message
-) new ly:basic-progress
-) get rid of the success function, use basic_progress instead
| Reinhold Kainhofer | 2011-09-03 | 2 | -3/+3 |
* | Fixes issue 1328....Finds the local minimum or maximum of a Bezier curve along the X
axis in the range of an intersection and shifts intersecting
curves up over this range plus slur padding.
Adds default slur padding to all scripts, as this algorithm makes
them almost just touch with the slur whereas the previous one already
had a bit of padding built into it because it was a generous approximation.
| Mike Solomon | 2011-08-25 | 2 | -0/+27 |
* | Loglevels: Document distinction between PROGRESS and INFO | Reinhold Kainhofer | 2011-08-19 | 2 | -6/+2 |
* | Get rid of some compiler warnings; Fix chdir handling of errors...-) Some "unused parameter" warnings
-) Some "ignoring return value" are a bit trickier. They are bad coding style:
o) the file-name.cc one returns a string with undefined contents if getcwd
fails, which might either crash guile or lead to other bugs.
o) the lily-parser-scheme.cc warning was also a programming error (if you
used --output=dir/file, and dir/ had the wrong permissions, then
lilypond would not print an error, but simply put the output file into
the current directory without telling the user!).
In this case (output dir explicitly requested, but not possible
to change there), I think it's best to exit lilypond with an
error.
-) signed/unsigned warning in glissando-engraver.cc, where we already
had a check for negative values, so explicitly casting to unsigned
shuts up the compiler.
-) Remove unneccessary #(set-paper-size "a6") in the regtests
-) fix bad texinfo code in function documentation
| Reinhold Kainhofer | 2011-08-14 | 1 | -3/+2 |
* | Proper loglevels: cmd-line option --loglevel=NONE/ERROR/WARN/BASIC/PROGRESS/D......Allow the user to specify which messages (s)he wants to see on stderr:
The lilypond code basically stays the same, I only added a loglevel
global variable, which is used in the warning/error*/progress*/success
functions (ly:warning, ly:error, etc. in Scheme). If the proper level
is not set for a message, it is not printed.
There are only some larger changes:
-) Global var be_verbose_global replaced by loglevel (in warn.cc, accessor
is_loglevel(...); much finer-grained)
-) New functions debug_output, which replaces code like:
if (be_verbose_global) {
progress_indication (...)
}
-) Move all scheme log functions to warn-scheme.cc
-) Add (optional) source location info to warning/error/log messages
All changes were done to warn.cc and to the member of the Input class
(apparently, we have two parallel error-reporting "frameworks" in
lilypond...).
Note for all functions in warn.cc (not for the members of Input):
The debug_output function (and progress_indication and message) have
an optional argument to specify whether the output of the message
should always start on a new line or continue the previous output.
All functions of the Input class now are just a front-end for the functions
in warn.cc
Documentation for this new feature is still missing (both in the AU
as well as in the CG).
| Reinhold Kainhofer | 2011-08-14 | 2 | -32/+168 |
* | Grand fixcc.py run on all .hh .cc files....Apologies for the inconvenience in patch handling, but getting
this done at once will cause less long-term problems than trying
to do this piecemeal.
Note for future git historians: this patch was created by running
scripts/auxiliar/fixcc.py \
$(find flower lily -name '*cc' -o -name '*hh' | grep -v /out)
with astyle 2.02 installed. No manual changes were made.
| Graham Percival | 2011-08-01 | 42 | -547/+532 |
* | Implement minmax() | Han-Wen Nienhuys | 2011-02-09 | 1 | -1/+7 |
* | Document Interval::delta(). | Han-Wen Nienhuys | 2011-01-31 | 1 | -0/+2 |
* | removed unused macro, function and file...The macro "classname" defined in flower/include/virtual-methods.hh is
never used. If it is removed then the function demangle_classname will
never be referenced, and hence file flower/rtti.cc can be removed.
I am assuming that they have not been added by someone who intends to use
them in future.
| Bernard Hurley | 2011-01-27 | 2 | -17/+0 |
* | Admin: run yearly grand-replace. | Graham Percival | 2011-01-13 | 42 | -42/+42 |
* | [flower] Remove compiler warnings....This has been tested with g++ 4.5.0 (20100722).
| Werner Lemberg | 2010-11-03 | 3 | -8/+8 |
* | T1265 - Remove deprecation warnings when running with Guile V2 Replace scm_st... | Ian Hulin | 2010-10-19 | 1 | -70/+15 |
* | Fix drastic overestimation of staff heights....The outside-staff simulation in f530eeb5bba has severe problems
when there many outside-staff objects per bar. This patch
mitigates the problems by assuming that outside-staff objects
don't intersect with each other (but only with the staff).
| Joe Neeman | 2010-10-01 | 2 | -0/+10 |
* | Refactor pure-height calculations. | Joe Neeman | 2010-09-28 | 2 | -0/+18 |
* | Clarify infinity_f preprocessor definition....The old code confuses various style programs, including our own
fixcc.py (it produces un-compilable code!). In particular:
-----
const Real infinity_f
= #ifdef
-----
which makes gcc unhapy.
The new format is easy for style programs to deal with, still
compiles, and is IMO just as clear anyway.
| Graham Percival | 2010-09-06 | 1 | -5/+3 |
* | Have LilyPond issue a success/failure termination message | Ian Hulin | 2010-05-08 | 2 | -0/+8 |
* | Configure: Remove unused header check. | Patrick McCarty | 2010-04-16 | 1 | -17/+0 |
* | Fix #887: Use ly:string-percent-encode for textedit URIs....* Add an overloaded instance of String_convert::bin2hex optimized for
converting single bytes to hex.
* Add a new callback, ly:string-percent-encode, to be used for percent
escaping paths in textedit URIs. This does the following:
- Leave unreserved characters in textedit URIs unescaped. This
includes 0-9, A-Z, a-z, and three punctuation characters (hyphen,
underscore, and full-stop).
- Leave the forward slash (/) unescaped, since it is used as a path
delimiter.
- Escape all other characters. Don't check for a null byte, since
those likely won't sneak into a full pathname.
* Use the callback function in the PS backend.
| Patrick McCarty | 2010-01-28 | 2 | -0/+10 |
* | Run grand-replace for 2010. | Neil Puttock | 2010-01-23 | 42 | -44/+60 |
* | Fix compilation issue on OS X | Nicolas Sceaux | 2009-12-21 | 1 | -7/+7 |
* | Update source file headers. Fixes using standard GNU package conventions....Signed-off-by: Han-Wen Nienhuys <hanwen@xs4all.nl>
Signed-off-by: Jan Nieuwenhuizen <janneke@gnu.org>
Signed-off-by: John Mandereau <john.mandereau@gmail.com>
Signed-off-by: Graham Percival <graham@percival-music.ca>
Signed-off-by: Reinhold Kainhofer <reinhold@kainhofer.com>
Signed-off-by: Joe Neeman <joeneeman@gmail.com>
Signed-off-by: Heikki Johannes Junes <hjunes@gmail.com>
Signed-off-by: Neil Puttock <n.puttock@gmail.com>
Signed-off-by: Carl D. Sorensen <c_sorensen@byu.edu>
Signed-off-by: Mats Bengtsson <mats.bengtsson@s3.kth.se>
Signed-off-by: Werner Lemberg <wl@gnu.org>
Signed-off-by: Patrick McCarty <pnorcks@gmail.com>
Signed-off-by: Nicolas Sceaux <nicolas.sceaux@free.fr>
| Jan Nieuwenhuizen | 2009-11-25 | 41 | -120/+582 |
* | Bump license to GPL v3+....Fixes future compatibility problems, GNU packages' policy defiance
and general confusion.
Signed-off-by: Han-Wen Nienhuys <hanwen@xs4all.nl>
Signed-off-by: Jan Nieuwenhuizen <janneke@gnu.org>
Signed-off-by: John Mandereau <john.mandereau@gmail.com>
Signed-off-by: Graham Percival <graham@percival-music.ca>
Signed-off-by: Reinhold Kainhofer <reinhold@kainhofer.com>
Signed-off-by: Joe Neeman <joeneeman@gmail.com>
Signed-off-by: Heikki Johannes Junes <hjunes@gmail.com>
Signed-off-by: Neil Puttock <n.puttock@gmail.com>
Signed-off-by: Carl D. Sorensen <c_sorensen@byu.edu>
Signed-off-by: Mats Bengtsson <mats.bengtsson@s3.kth.se>
Signed-off-by: Werner Lemberg <wl@gnu.org>
Signed-off-by: Patrick McCarty <pnorcks@gmail.com>
Signed-off-by: Nicolas Sceaux <nicolas.sceaux@free.fr>
| Jan Nieuwenhuizen | 2009-11-25 | 2 | -6/+2 |
* | Fix possible endless loop in replace_all...Inside replace all, start the search for the next appearance from the
end position of the replacement, not from the end of the occurrence.
Otherwise, replace_all (&str, "\"", "\\\"") will result in an endless
loop.
| Reinhold Kainhofer | 2009-08-22 | 1 | -1/+2 |
* | Fix a thinko that broke the contract of STL sort. | Joe Neeman | 2009-07-24 | 1 | -1/+1 |
* | Typos: Lilypond -> LilyPond, explicite -> explicit, fix docstring | Mark Polesky | 2009-06-02 | 2 | -2/+2 |
* | Run grand-replace for 2009. | Jan Nieuwenhuizen | 2009-01-06 | 40 | -40/+40 |
* | Remove SCons traces because of unmaintained state | John Mandereau | 2009-01-05 | 1 | -28/+0 |
* | More C++. | Werner Lemberg | 2008-12-27 | 2 | -15/+9 |
* | Run `make grand-replace'. | Jan Nieuwenhuizen | 2008-12-01 | 40 | -40/+40 |
* | Define our own U64_MAX maximum. | Han-Wen Nienhuys | 2008-09-18 | 3 | -1/+13 |
* | Use long long integers in Rational class....Use typedef long long from flower-proto.hh.
| Neil Puttock | 2008-09-12 | 5 | -23/+41 |
* | Interface changes for string functions:... - replace_all() takes string*
- remove some scm_i_XXX uses.
| Han-Wen Nienhuys | 2008-06-01 | 4 | -14/+14 |
* | Add Drul_array<>::set() method. | Han-Wen Nienhuys | 2008-05-01 | 1 | -1/+5 |