Hi Carl,
LGTM.  I do have one comment about tests for != SCM_UNDEFINED.  It think
there's less chance of confusion when testing for "undefined" rather
than "not undefined", which is "defined".  The double negative (not
undefined) can potentially be confusing.
that's a good point. However, I'm going to be on vacation for a week and will rewrite this when I come back.

Regards,
Michael



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

Reply via email to