|
From: | janek . lilypond |
Subject: | Re: note-spacing: stretch somewhat uniformly; issue 3304 (issue 36830045) |
Date: | Sun, 29 Dec 2013 20:59:28 +0000 |
https://codereview.appspot.com/36830045/diff/120001/lily/ledger-line-spanner.cc File lily/ledger-line-spanner.cc (right): https://codereview.appspot.com/36830045/diff/120001/lily/ledger-line-spanner.cc#newcode103 lily/ledger-line-spanner.cc:103: if (Staff_symbol::ledger_positions (staff, pos).empty ()) Nice catch! Please add a regtest for this. Since the regtest will concern a very minute difference, i suggest to use staffsize ~50-80 to draw reviewer's attention. https://codereview.appspot.com/36830045/
[Prev in Thread] | Current Thread | [Next in Thread] |