|
From: | lilypond |
Subject: | Re: [Lilypond-auto] Issue 2823 in lilypond: Patch: Fixes position of mensural c clef |
Date: | Sun, 09 Sep 2012 17:06:04 +0000 |
Comment #18 on issue 2823 by address@hidden: Patch: Fixes position of mensural c clef
http://code.google.com/p/lilypond/issues/detail?id=2823The 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.)
[Prev in Thread] | Current Thread | [Next in Thread] |