|
| From: | k-ohara5a5a |
| Subject: | Re: Issue 3641: Keep only one Axis_group_engraver active (issue 20500043) |
| Date: | Fri, 01 Nov 2013 07:30:48 +0000 |
Looks good. https://codereview.appspot.com/20500043/diff/20001/lily/axis-group-engraver.cc File lily/axis-group-engraver.cc (right): https://codereview.appspot.com/20500043/diff/20001/lily/axis-group-engraver.cc#newcode135 lily/axis-group-engraver.cc:135: the context. */ Probably this comment no longer applies. https://codereview.appspot.com/20500043/diff/20001/lily/axis-group-engraver.cc#newcode147 lily/axis-group-engraver.cc:147: && staffline_->get_parent (Y_AXIS) == elts_[i]) Maybe this test is no longer needed. https://codereview.appspot.com/20500043/
| [Prev in Thread] | Current Thread | [Next in Thread] |