|
From: | lilypond |
Subject: | Re: stem.cc - issue 5303 - misplaced notehead (issue 570830043 by address@hidden) |
Date: | Sat, 06 Jul 2019 01:06:43 -0700 |
Maybe the snipped should be renamed also. https://codereview.appspot.com/570830043/diff/578780043/input/regression/misplaced-note-head-bug.ly File input/regression/misplaced-note-head-bug.ly (right): https://codereview.appspot.com/570830043/diff/578780043/input/regression/misplaced-note-head-bug.ly#newcode1 input/regression/misplaced-note-head-bug.ly:1: \version "2.19.16" IIUC this should be the first version a feature is introduced or a bug is fixed, i. e. 2.21.0 https://codereview.appspot.com/570830043/diff/578780043/input/regression/misplaced-note-head-bug.ly#newcode5 input/regression/misplaced-note-head-bug.ly:5: texidoc = "Misplaced note head bug (issue 5303) should be fixed." Maybe that comment can be more descriptive so that you don’t have to search for that issue to understand it? Something like “Note heads are placed on the correct side of the stem; layout-set-staff-size doesn’t change that placement.” https://codereview.appspot.com/570830043/
[Prev in Thread] | Current Thread | [Next in Thread] |