[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Get rid of some compiler warnings (issue4854049)

From: k-ohara5a5a
Subject: Re: Get rid of some compiler warnings (issue4854049)
Date: Wed, 10 Aug 2011 04:39:24 +0000

Reinhold Kainhofer <reinhold <at>> writes:

Am Freitag, 5. August 2011, 21:22:49 schrieb Keith OHara:

> The entire source tree generates only seven warnings, all but one of
> point to code we could write more clearly.

Ah, great. Will you try to fix them?

You beat me to it :-)
I'll clean up the .ll, unless Neil has already done that.
File lily/ (right):
lily/ if (n1 < 0 || n2 < 0 || unsigned(n1) >=
note_heads.size ())
just confirming, that was what the compiler was doing, per the C++
File lily/ (right):
lily/ // FIXME: This has hardcoded logic, assuming a ==
 assert (AXIS_X == a);
File lily/ (right):
Lyric_combine_music_iterator::check_new_context (SCM /*sev*/)
On 2011/08/09 21:20:29, Carl wrote:
Why not /*SCM sev*/ with a FIXME?

the function is declared with DECLARE_LISTENTER so it has to take a
stream event, even if it doesn't use it.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]