[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Eliminates the Hara_kiri_engraver. (issue 7061062)
From: |
k-ohara5a5a |
Subject: |
Re: Eliminates the Hara_kiri_engraver. (issue 7061062) |
Date: |
Mon, 14 Jan 2013 18:20:44 +0000 |
https://codereview.appspot.com/7061062/diff/1/lily/axis-group-engraver.cc
File lily/axis-group-engraver.cc (right):
https://codereview.appspot.com/7061062/diff/1/lily/axis-group-engraver.cc#newcode122
lily/axis-group-engraver.cc:122: {
On 2013/01/14 17:50:47, Keith wrote:
The property haraKiri [...] is now redundant with
keepAliveInterfaces.
Or with remove-empty, I suppose.
If you can access the value with
staffline_ && to_boolean(staffline_ -> get_property("remove-empty"))
at this point, that would seem to be simpler.
https://codereview.appspot.com/7061062/
- Eliminates the Hara_kiri_engraver. (issue 7061062), mtsolo, 2013/01/10
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), dak, 2013/01/14
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), k-ohara5a5a, 2013/01/14
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062),
k-ohara5a5a <=
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), dak, 2013/01/14
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), dak, 2013/01/20
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), mtsolo, 2013/01/20
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), k-ohara5a5a, 2013/01/21
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), k-ohara5a5a, 2013/01/21
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), k-ohara5a5a, 2013/01/21
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), k-ohara5a5a, 2013/01/25
- Re: Eliminates the Hara_kiri_engraver. (issue 7061062), dak, 2013/01/25