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