Hello, Is this patch correct?
The reason I ask is because, after applying this patch, the results of `make check' show many spacing changes, and I don't know if those changes occur because the patch is incorrect or if there are spacing-related workarounds in the codebase that need to be adjusted. Also, the new regtest `stem-length-estimation.ly' will need to be reworked to fit all systems on one page again, and I'll take of that if this patch is okay. One example of the spacing changes is attached. Thanks, Patrick
>From e45c647771360e6809c437c98ca6ea7bd54a99fe Mon Sep 17 00:00:00 2001 From: Patrick McCarty <[email protected]> Date: Sun, 23 May 2010 13:30:06 -0700 Subject: [PATCH] Initialize interval in grob_stencil_extent(). There is a chance of returning garbage values from grob_stencil_extent() due to this uninitialized variable issue. --- lily/grob.cc | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lily/grob.cc b/lily/grob.cc index 911243b..d5d5309 100644 --- a/lily/grob.cc +++ b/lily/grob.cc @@ -682,7 +682,7 @@ static SCM grob_stencil_extent (Grob *me, Axis a) { Stencil *m = me->get_stencil (); - Interval e; + Interval e (0, 0); if (m) e = m->extent (a); return ly_interval2scm (e); -- 1.7.1
<<attachment: song-melisma.png>>
<<attachment: song-melisma.compare.jpeg>>
_______________________________________________ lilypond-devel mailing list [email protected] http://lists.gnu.org/mailman/listinfo/lilypond-devel
