lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Gets vertical skylines from grob stencils (issue 5626052)


From: Joe Neeman
Subject: Re: Gets vertical skylines from grob stencils (issue 5626052)
Date: Mon, 6 Feb 2012 11:00:55 -0800

Could you publicize work-in-progress patches like this as a git branch instead of a Rietveld issue? Git branches are easy for reviewers to check out, fast-forward, revert, etc. That way, people can offer broad comments. When the patch is "finished" (ie. you can't think of anything you need to change), then upload it to rietveld, where it's easy to make specific, in-line comments.

Avoiding rietveld until the patch is done will also prevent the situation where someone makes a bunch of detailed comments to lines that you're in the middle of changing anyway.

Cheers,
Joe

On Sat, Feb 4, 2012 at 1:28 PM, <address@hidden> wrote:
Reviewers: ,

Message:
Sorry in advance for the whitespace errors.

This patch provides a generic framework for building vertical skylines
out of boxes by traversing through a stencil.  It currently only
implements skylines for three stencil types (draw-line, named-glyph, and
glyph-string) and can do more for other glyphs.  It is modular, so as
skyline estimation improve for a given stencil, that function can be
worked on alone w/o touching the other parts of stencil-integral.scm.

The implementation for glyph-string is ugly and slightly incorrect for y
heights of these stencils...it would be infinitely easier if the stencil
could contain information about y extents, but unfortunately,
PangoGlyphGeometry does not carry height information.  This error does
not have a bearing on the visual results in this patch, but it'd be good
to find a way to make the dimensions exact.

I'm also not sure that line thickness is tacked on correctly for the
draw-line boxes, but it seems to yield correct results.

Cheers,
MS

Description:
Gets vertical skylines from grob stencils

Please review this at http://codereview.appspot.com/5626052/

Affected files:
 A input/regression/tuplet-bracket-vertical-skylines.ly
 A input/regression/volta-bracket-vertical-skylines.ly
 M lily/axis-group-interface.cc
 A lily/box-scheme.cc
 M lily/include/box.hh
 M lily/include/skyline.hh
 M lily/skyline-pair.cc
 A lily/skyline-scheme.cc
 M lily/skyline.cc
 M lily/stencil-scheme.cc
 M lily/tuplet-bracket.cc
 M lily/volta-bracket.cc
 M scm/define-grobs.scm
 M scm/lily.scm
 A scm/stencil-integral.scm



_______________________________________________
lilypond-devel mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/lilypond-devel


reply via email to

[Prev in Thread] Current Thread [Next in Thread]