lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Fixes position of mensural c clef (issue 6503091)


From: lemzwerg
Subject: Re: Fixes position of mensural c clef (issue 6503091)
Date: Mon, 24 Sep 2012 17:34:36 +0000

The glyph shape is OK, thanks.  Have you tested the appearance on paper
of all available sizes, using a 300 or 600dpi printer?


http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf
File mf/parmesan-clefs.mf (left):

http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf#oldcode902
mf/parmesan-clefs.mf:902:
The dimension of the bounding box is not correct.  It's far too large,
as can be seen in the DVI proof.

http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf
File mf/parmesan-clefs.mf (right):

http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf#newcode791
mf/parmesan-clefs.mf:791: ..z2+(0, blot_rad)
This must be `.. {up}z2 + (0, blot_rad)', otherwise the curve doesn't
start with the right direction.  This wouldn't be that bad, but it were
different to all other glyph shapes.

Ditto for all other transitions from `--' to `..' (this is, using the
proper `{up}', `{right}', etc.).

http://codereview.appspot.com/6503091/diff/16001/mf/parmesan-clefs.mf#newcode793
mf/parmesan-clefs.mf:793: --quartercircle scaled blot_diameter rotated
180 shifted (z3r + (blot_rad, blot_rad))
Using `quartercircle' gives good results in the type1 output.  I only
ask you to make the line fit into 80 columns :-)

On the other hand, why not simply using something like

  startcurve_point{down} .. {right}endcurve_point

?  This might be shorter, and easier to read.

http://codereview.appspot.com/6503091/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]