Reviewers: ,
Description:
Add RemoveEmptyStavesAll command
It has been noted several times that we should have a companion
for RemoveEmptyStaves to also set remove-first to ##t. I’ve
made an attempt at providing it – please review. A doc snippet,
removing-the-first-empty-line.ly, has become ob
The name is rather clumsy (and it does not help that the issue report
talks about "\RemoveEmptyStavesFirst" while the actual patch and the
Rietveld review has "\RemoveEmptyStavesAll").
How about "\RemoveAllEmptyStaves" ?
https://codereview.appspot.com/283050043/
On 14.01.2016 16:20, d...@gnu.org wrote:
The name is rather clumsy (and it does not help that the issue report
talks about "\RemoveEmptyStavesFirst" while the actual patch and the
Rietveld review has "\RemoveEmptyStavesAll").
That’s already fixed.
How about "\RemoveAllEmptyStaves" ?
Vote t
Am 14.01.2016 um 16:34 schrieb Simon Albrecht:
> On 14.01.2016 16:20, d...@gnu.org wrote:
>> The name is rather clumsy (and it does not help that the issue report
>> talks about "\RemoveEmptyStavesFirst" while the actual patch and the
>> Rietveld review has "\RemoveEmptyStavesAll").
>
> That’s al
Simon Albrecht wrote Thursday, January 14, 2016 3:34 PM
> On 14.01.2016 16:20, d...@gnu.org wrote:
>> The name is rather clumsy (and it does not help that the issue report
>> talks about "\RemoveEmptyStavesFirst" while the actual patch and the
>> Rietveld review has "\RemoveEmptyStavesAll").
>>
"Trevor Daniels" writes:
> Simon Albrecht wrote Thursday, January 14, 2016 3:34 PM
>
>
>> On 14.01.2016 16:20, d...@gnu.org wrote:
>>> The name is rather clumsy (and it does not help that the issue report
>>> talks about "\RemoveEmptyStavesFirst" while the actual patch and the
>>> Rietveld review
Renamed the new command according to the vote.
https://codereview.appspot.com/283050043/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
