Re: Create engravers for merging rests (issue 321930043 by horndud...@gmail.com)

2017-05-19 Thread horndude77
https://codereview.appspot.com/321930043/diff/60001/ly/init.ly File ly/init.ly (right): https://codereview.appspot.com/321930043/diff/60001/ly/init.ly#newcode36 ly/init.ly:36: #(use-modules (scm merge-rests-engraver)) On 2017/05/18 14:15:23, david.nalesnik wrote: I'm not sure why you are defini

Re: Create engravers for merging rests (issue 321930043 by horndud...@gmail.com)

2017-05-20 Thread horndude77
https://codereview.appspot.com/321930043/diff/11/scm/scheme-engravers.scm File scm/scheme-engravers.scm (right): https://codereview.appspot.com/321930043/diff/11/scm/scheme-engravers.scm#newcode151 scm/scheme-engravers.scm:151: (define (rest-eqv rest-len-prop) On 2017/05/20 12:18:33, tho

Re: Create engravers for merging rests (issue 321930043 by horndud...@gmail.com)

2017-05-27 Thread horndude77
On 2017/05/21 17:12:26, thomasmorley651 wrote: > I'd like to mention another point: > What to do with pitched rests and rests with user-set staff-position, merge them > automatically to the zero-position? If a user has explicitly set the position of a rest this should be honoured by default