Etude

2011-05-08 Thread Simon Mackenzie
Hi Daniel. Your app looks interesting and I was reading you email to lilypond-devel and wondering if you would be willing to make an addition, if you are serious about supporting the lilypond project that is. ;-) Why not release into the public domain a tidied up version of your "compiler" and a

RE: autoconf not mentioned in build dependencies

2011-05-08 Thread James Lowe
Hello, From: lilypond-devel-bounces+james.lowe=datacore@gnu.org [lilypond-devel-bounces+james.lowe=datacore@gnu.org] on behalf of Jan Nieuwenhuizen [jann...@gnu.org] Sent: 08 May 2011 09:57 To: Andrew Hawryluk Cc: lilypond-devel Subject: Re: autoco

Re: Bugfix for issue 1630 (issue4490045)

2011-05-08 Thread tdanielsmusic
LGTM, except for non-standard indentation which should be corrected, although I've tested it only on the examples in the regression tests. Trevor http://codereview.appspot.com/4490045/diff/1/lily/completion-note-heads-engraver.cc File lily/completion-note-heads-engraver.cc (right): http://cod

Re: autoconf not mentioned in build dependencies

2011-05-08 Thread Jan Nieuwenhuizen
Andrew Hawryluk writes: > I'm building from scratch on a new Ubuntu install and autoconf is not > mentioned as a requirement: > http://lilypond.org/doc/v2.13/Documentation/contributor/requirements-for-compiling-lilypond Autoconf is only needed when building from GIT. The tarball includes ./confi

Re: Fix calculation of vertical offset when 'staff-padding is set (issue4489042)

2011-05-08 Thread Trevor Daniels
lemniskata.bernoull...@gmail.com wrote Sunday, May 08, 2011 7:17 AM Style nitpick. http://codereview.appspot.com/4489042/diff/1/lily/side-position-interface.cc#newcode65 lily/side-position-interface.cc:65: bool include_staff = I see quite a lot of whitespace diffs here and there. IIRC we ca