Thanks for the review, Janek.
http://codereview.appspot.com/4654084/diff/42/mf/feta-flags-generic.mf File mf/feta-flags-generic.mf (right): http://codereview.appspot.com/4654084/diff/42/mf/feta-flags-generic.mf#newcode4 mf/feta-flags-generic.mf:4: % Copyright (C) 1997--2011 Han-Wen Nienhuys <[email protected]> On 2011/07/04 20:19:31, Janek Warchol wrote:
Shouldn't your name be here, Carl?
I think possibly it should be added, but mostly I just copied Han-Wen's work. http://codereview.appspot.com/4654084/diff/42/mf/feta-scripts.mf File mf/feta-scripts.mf (right): http://codereview.appspot.com/4654084/diff/42/mf/feta-scripts.mf#newcode1450 mf/feta-scripts.mf:1450: begingroup; On 2011/07/04 20:19:31, Janek Warchol wrote:
You added this because every glyph should be in some group now?
No, the begingroup and endgroup define the scope over which the save functions. and the height, overshoot, and width that are used in this glyph should be limited to this glyph. http://codereview.appspot.com/4654084/diff/42/mf/feta-scripts.mf#newcode1452 mf/feta-scripts.mf:1452: height# := staff_space#; On 2011/07/04 20:19:31, Janek Warchol wrote:
are the colons only a matter of style?
No. The colon makes it an assignment. Without the colon, it's an equation that becomes a constraint; eventually the equation is solved. The intention here, AFAICS, is to have it be an assignment, so I made it explicit. http://codereview.appspot.com/4654084/ _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
