Comment #18 on issue 2823 by [email protected]: Patch: Fixes position of mensural c clef
http://code.google.com/p/lilypond/issues/detail?id=2823

The very problem with your code changes is that they don't work properly for low-resolution devices. Shifting by non-integer pixel values produces unpredictable and usually unsymmetrical results. This is also the very reason that pseudo-operators like `lft' and `top' are used, namely to align integer coordinates values so that you get uniform line thickness.

Admittedly, contrary to the Feta glyphs, I haven't tested the Parmesan glyphs this way, and today it is probably of no importance because MetaFont no longer rasterizes directly to B/W devices. But for consistency I really ask you to follow the conventions of the remaining MF code, adding comments where necessary. (BTW, the comment character `%' should be immediately precede by a whitespace character.)



Reply via email to