http://codereview.appspot.com/6308093/diff/1/lily/self-alignment-interface.cc
File lily/self-alignment-interface.cc (right):

http://codereview.appspot.com/6308093/diff/1/lily/self-alignment-interface.cc#newcode206
lily/self-alignment-interface.cc:206: grob_alignment = scm_to_double
(scm_cdr (grob_alignment_property));
Use robust_scm2double

http://codereview.appspot.com/6308093/diff/1/lily/self-alignment-interface.cc#newcode221
lily/self-alignment-interface.cc:221: : me->core_extent (me, a);
What about:
if (which_grob_extent == ly_symbol2scm ("extent"))
  grob_extent = me->extent (me, a);
else
  grob_extent = ly_scm2interval(me->get_property (which_grob_extent);

Then you can get rid of all the core-extent-related caching code.

http://codereview.appspot.com/6308093/

_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to