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

2017-05-18 Thread thomasmorley65
https://codereview.appspot.com/321930043/diff/60001/scm/merge-rests-engraver.scm File scm/merge-rests-engraver.scm (right): https://codereview.appspot.com/321930043/diff/60001/scm/merge-rests-engraver.scm#newcode10 scm/merge-rests-engraver.scm:10: (define (rest-length rest) This definition is

Re: [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #1388 Support OpenType font features

2017-05-18 Thread tisimst
On Thu, May 18, 2017 at 2:34 PM, Xavier Scheuer [via Lilypond] < ml+s1069038n203182...@n5.nabble.com> wrote: > On 18 May 2017 at 07:35, Auto mailings of changes to Lily Issues < > [hidden email] > > wrote: > > > > Initial work to support opentype

Re: Calling in for sickness

2017-05-18 Thread Paul
Hi David, I'm really sorry to hear your news and I hope your health improves. All my best, -Paul ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #1388 Support OpenType font features

2017-05-18 Thread Xavier Scheuer
On 18 May 2017 at 07:35, Auto mailings of changes to Lily Issues < testlilyissues-a...@lists.sourceforge.net> wrote: > > Initial work to support opentype font features > > Following up on an old patch I send to the list. One comment was to use a list > for combining multiple font features. This is

Re: Merge Rests Engraver

2017-05-18 Thread Xavier Scheuer
On 14 May 2017 at 06:15, Jay Anderson wrote: > > https://codereview.appspot.com/321930043/ > > It was requested sometime last year that I attempt to get this into > lilypond proper this is my attempt. It creates a scheme module for putting > all of the merge rest

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

2017-05-18 Thread david . nalesnik
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)) I'm not sure why you are defining a separate module. The usual procedure

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

2017-05-18 Thread pkx166h
Passes make, make check and a full make doc. https://codereview.appspot.com/321930043/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel