Hi Carl,

Are you sure this is ready for review?

I see comments like this,

// I removed this to solve a bug; need to make sure it's right -- cds

and commented debug code,

+;(display "\nIn auto-beam.scm\n")

and alarm bells start to ring. :)

Also, there are several places where beamSettings is capitalized (most
importantly, in bagpipe.ly.)

BTW, what happens if a time signature has no default settings?

Cheers,
Neil

http://codereview.appspot.com/1667041/show

_______________________________________________
lilypond-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to