On Sun, Aug 29, 2010 at 3:54 PM, Jan Nieuwenhuizen <janneke-l...@xs4all.nl> wrote: > Op zondag 29-08-2010 om 18:04 uur [tijdzone +0000], schreef > hanw...@gmail.com: > >> http://codereview.appspot.com/1579041/diff/42001/43004#newcode109 >> lily/metronome-engraver.cc:109: } >> this is out of style with the rest of lilypond code base. >> >> The normal pattern is to use XXx_interface::has_interface (for >> hard-coded interfaces), or grob->[internal_]has_interface(), to check >> for softcoded interfaces. > > Thanks for catching this. I've applied the patch below.
LGTM -- Han-Wen Nienhuys - han...@xs4all.nl - http://www.xs4all.nl/~hanwen _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel
