Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1817)

Issue 7762049: Gives invisible stems point extents. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 1 month ago by MikeSol
Modified:
11 years, 1 month ago
Reviewers:
Keith, mike7
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Gives invisible stems point extents.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M lily/stem.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2
Keith
So then the convention for an extent is '(+inf.0 . -inf.0) is empty, but for ...
11 years, 1 month ago (2013-03-14 07:05:47 UTC) #1
mike7
11 years, 1 month ago (2013-03-14 10:28:08 UTC) #2
On 14 mars 2013, at 08:05, k-ohara5a5a@oco.net wrote:

> So then the convention for an extent is
> '(+inf.0 . -inf.0) is empty, but for skylines in the cross direction it
> is an infinite wall;
> '(0 . 0) is a point, but ignored for skylines;
> '(-inf.0 . +inf.0) is an infinite wall all the time.
> 
> Could you filter out this effect on RhythmicStaves ?
> 
> My heavy-metal band uses a flame-shaped glyph for some Stems, and it
> needed zero X-extent for some spacing reason.  While fixing another MIDI
> bug I had the Midi_performer listen for Stems and send a message, if on
> channel 10, when it sees a zero-width Stem, by convention.  That message
> fires the flamethrowers, so if you push this change I have to remind the
> boys to avoid that part of the stage in case the drummer has any beamed
> rests.
> 
> https://codereview.appspot.com/7762049/

New patch up that doesn't get in the way of your band!

Cheers,
MS
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b