[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Removes kludges in stencil-based skyline code. (issue 7516048)
|
From: |
k-ohara5a5a |
|
Subject: |
Re: Removes kludges in stencil-based skyline code. (issue 7516048) |
|
Date: |
Sat, 30 Mar 2013 04:52:13 +0000 |
https://codereview.appspot.com/7516048/diff/13001/lily/side-position-interface.cc
File lily/side-position-interface.cc (left):
https://codereview.appspot.com/7516048/diff/13001/lily/side-position-interface.cc#oldcode317
lily/side-position-interface.cc:317: // this seems kinda kludgy, as
there is no apparent logic to it
Probably not so kludgy if we requested staff-padding on something placed
in a Dynamics context.
I suggest that in if (include_staff) {} above we make staff_extents at
least the interval (0 . 0)
https://codereview.appspot.com/7516048/