On Aug 25, 2011, at 3:17 PM, Han-Wen Nienhuys wrote: > On Thu, Aug 25, 2011 at 7:03 AM, Mike Solomon <[email protected]> wrote: > >> I have verified via pacifier prints that the pure heights of the flag are >> being taken from the stencil function and are going into the spacing engine >> in separation-item.cc. However, in theory, this patch should have null >> effect on the minimum and ideal distances of paper columns. This is not the >> case (see attached). > > ? it's not the heights but the widths that should go into separation-item? > > Have you verified that the new flag grob gets taken into account in > note-spacing.cc ? >
I found the problem after a couple hours of banging my head against the wall. The Stem::width function uses Stem::flag to get the flag, whereas the stencil uses Stem::get_translated_flag. Thus, the flag used for widths is not translated. As the flag's bounding box in the font is shifted by a half stem length to the right of where the glyph actually is, and as the translated flag was always shifted by half a stem width, this is what made it work (albeit kludgily). I'll work this into the patch tonight or tomorrow. Cheers, MS _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
