bug-lilypond
[Top][All Lists]
Advanced

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

Re: Issue 2109 in lilypond: do not tinker with the position of a pitched


From: lilypond
Subject: Re: Issue 2109 in lilypond: do not tinker with the position of a pitched rest
Date: Thu, 02 Feb 2012 18:07:10 +0000


Comment #14 on issue 2109 by address@hidden: do not tinker with the position of a pitched rest
http://code.google.com/p/lilypond/issues/detail?id=2109

Assuming the input code for comment 13 was
  \new Staff \with {
    \override StaffSymbol #'line-count = 6  % also 5, and 7
  } \relative f { \clef alto b2 b\rest c c\rest d d\rest e e\rest }

old LilyPond placed pitches rests correctly only when the pitch lies on a line in a default staff, varying the relation between pitch- and rest-location.

new LilyPond places them in a constant relation to the pitch, so they are correct when the pitch lies on a line in the staff actually being drawn

The new behavior is probably a bit easier to use.

Problems
1) The regression test added by the patch, however, produces recognizable rests with the old code, and what appear to be errors with the new code. Pál, pleas change this so the regression test shows desired output. Otherwise, we will be confused in six months and might reverse your change trying to 'fix' the output of the regtest.

2) The bug report gives no description of the desired output after the change, just a link to a patch. I think the bug squad is asked to mark these Validated as soon as the change appears in a release, without reviewing the change. Likewise I don't bother reviewing the code in these cases.


reply via email to

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