-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On Samstag, 11. April 2009 20:33:47 n.putt...@gmail.com wrote: > I never realized this would be so simple, but it strikes me as a bit of > a hack. > > In your sample text dynamic spanners, there seems to be an element of > redundancy in the event properties; you could easily junk > '(de)crescendoSpanner and '(de)crescendoText, using 'text only to > trigger the change,
Yes, but I decided against it, since the global setting shows the same redundancy. \set #'crescendoSpanner = #'text \set #'crescendoText = #"blah" I wanted to stay consistent for global and for one-time settings. However, I totally agree that this is redundant, so I would also not mind junking the global crescendoSpanner as well (although it gives us flexibility should we want to introduce other spanner types than hairpin and text). > http://codereview.appspot.com/39047/diff/1/2 > File lily/new-dynamic-engraver.cc (right): > > http://codereview.appspot.com/39047/diff/1/2#newcode76 > Line 76: SCM spanner_type = evt->get_property (name); > This looks weird to me, since you're taking advantage of the fact that > the event properties you're setting look like context properties. No, I simply check if the event property with the same name exists. If it does, it takes precedence. What I'm taking advantage of that I assume that the event property and the global setting have the same name. > You'd > need a method which does the reverse of camel_case_to_lisp_identifier () > so that the events would be converted from e.g. crescendo-spanner -> > crescendoSpanner. Huh? Sorry, but I don't understand what you want to say here. Can you please elaborate? Cheers, Reinhold - -- - ------------------------------------------------------------------ Reinhold Kainhofer, reinh...@kainhofer.com, http://reinhold.kainhofer.com/ * Financial & Actuarial Math., Vienna Univ. of Technology, Austria * http://www.fam.tuwien.ac.at/, DVR: 0005886 * LilyPond, Music typesetting, http://www.lilypond.org -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) iD8DBQFJ4Rl3TqjEwhXvPN0RApgwAKDC/jJxn69zQgLUklKCviFbiFAvKwCgs3Ha eeq0/YPhHKEMCNZLqDgT32Y= =lOSc -----END PGP SIGNATURE----- _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel