Updates:
Status: Verified
Comment #21 on issue 804 by philehol...@googlemail.com: Code cleaning:
checking types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Ran make on my 32-bit VM and only get:
out/parser.cc:2407: warning:
Comment #22 on issue 804 by d...@gnu.org: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
out/parser.cc:2407: warning: conversion to 'short int' from 'int' may
alter its value
I think I once tried tracking
Updates:
Status: Fixed
Comment #20 on issue 804 by reinhold...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
The 64-bit warnings are now in a separate bug report (issue 1890), all
other
Updates:
Status: Verified
Comment #15 on issue 804 by brownian.box: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
(No comment was entered for this change.)
Updates:
Status: Accepted
Comment #16 on issue 804 by reinhold...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
That patch only fixes all warnings on 32 bit systems. Purportedly (and
that's
Comment #17 on issue 804 by brownian.box: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Sorry, I've misunderstood your comment (c#14).
___
bug-lilypond mailing list
Comment #18 on issue 804 by reinhold...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Comment #14 was about that one last warning on 32-bit systems mentioned in
comment #13. Since I don't have a
Comment #19 on issue 804 by k-ohara5...@oco.net: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
I would not find the further changes of patch 4 to be helpful. The next
step on this issue, I think, is to wait
Comment #14 on issue 804 by reinhold...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Yes, I'd leave that warning in. It is a minor issue, which we choose to
ignore for now, so we shouldn't
Updates:
Status: Fixed
Labels: -Patch-review
Comment #13 on issue 804 by k-ohara5...@oco.net: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Lexer warnings resolved with commit da949cdc
On
Comment #11 on issue 804 by percival.music.ca: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Reinhold has pushed his patch; the patch under review if Keith's changes to
the lexer.
Updates:
Labels: -Patch-review Patch-needs_work
Comment #9 on issue 804 by colinpkc...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Per the Rietveld issue, Keith is still revising this.
Updates:
Labels: -Patch-needs_work Patch-review
Comment #10 on issue 804 by k-ohara5...@oco.net: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Keith is still revising this.
No, I just made commentary
Comment #7 on issue 804 by reinhold...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
These regtest differences are there because previously the
(set-paper-size a6) calls didn't have any effect, as
Comment #8 on issue 804 by reinhold...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Another patch by Keith to fix the lexer warnings on 32-bit linux:
http://codereview.appspot.com/4871041/
Updates:
Labels: -Patch-new Patch-review
Comment #6 on issue 804 by pkx1...@gmail.com: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Passes make but there are some reg test differences - see attached
Updates:
Labels: -Patch Patch-new
Comment #5 on issue 804 by percival.music.ca: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
The previous patch does not apply cleanly to master.
Fortunately, Reinhold
Comment #3 on issue 804 by n.puttock: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
Issue 1144 has been merged into this issue.
___
bug-lilypond mailing list
Updates:
Labels: Patch
Comment #4 on issue 804 by n.puttock: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
http://codereview.appspot.com/1724041/show
___
Updates:
Summary: Code cleaning: checking types-conversion issues and other build
warnings
Owner: ---
Comment #2 on issue 804 by percival.music.ca: Code cleaning: checking
types-conversion issues and other build warnings
http://code.google.com/p/lilypond/issues/detail?id=804
20 matches
Mail list logo