Re: absolute pitch entry: accept an offset octave (issue 235010043 by k-ohara5...@oco.net)

2015-05-02 Thread dak
On 2015/05/03 06:28:43, dak wrote: https://codereview.appspot.com/235010043/diff/1/ly/music-functions-init.ly File ly/music-functions-init.ly (right): https://codereview.appspot.com/235010043/diff/1/ly/music-functions-init.ly#newcode36 ly/music-functions-init.ly:36: ((integer?) ly:music?) Wh

absolute pitch entry: accept an offset octave (issue 235010043 by k-ohara5...@oco.net)

2015-05-02 Thread dak
https://codereview.appspot.com/235010043/diff/1/ly/music-functions-init.ly File ly/music-functions-init.ly (right): https://codereview.appspot.com/235010043/diff/1/ly/music-functions-init.ly#newcode36 ly/music-functions-init.ly:36: ((integer?) ly:music?) Why use a number here instead of a pitch?

Re: Partcombiner documentation (Issue 4307) (issue 233110043 by philehol...@googlemail.com)

2015-05-02 Thread k-ohara5a5a
Looks fine. https://codereview.appspot.com/233110043/diff/1/Documentation/notation/simultaneous.itely File Documentation/notation/simultaneous.itely (right): https://codereview.appspot.com/233110043/diff/1/Documentation/notation/simultaneous.itely#newcode937 Documentation/notation/simultaneous.

Re: stencil-flip function

2015-05-02 Thread Paul Morris
> On May 2, 2015, at 5:46 PM, Carl Sorensen wrote: > > I think you should submit this code as a patch to scm/stencil.scm. Also, > the regtests should be fixed to show the new behavior. > > Using a scale of -1 is the appropriate thing to do when you want to have > the *origin* stay in the same l

Re: Just some short feedback

2015-05-02 Thread Janek Warchoł
Hi David, thanks for this update! 2015-04-27 11:09 GMT+02:00 David Kastrup : > At any rate, I am starting over _again_ but I think that I am now at the > stage where my plan of execution is nicely streamlined and "Listeners" > from both C++ and Scheme level (as well as their creation from the > b

any LilyPonders who'd like to meet in California?

2015-05-02 Thread Janek Warchoł
Hello, I'll be in San Francisco area for the next three weeks - anyone would like to meet? It would be great :) best, Janek ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: stencil-flip function

2015-05-02 Thread Janek Warchoł
2015-05-02 23:46 GMT+02:00 Carl Sorensen : > I think you should submit this code as a patch to scm/stencil.scm. Also, > the regtests should be fixed to show the new behavior. > > Using a scale of -1 is the appropriate thing to do when you want to have > the *origin* stay in the same location. Usi

Re: stencil-flip function

2015-05-02 Thread Carl Sorensen
On 5/2/15 2:45 PM, "Paul Morris" wrote: >Hello dev list, > >Recently while working on Jianpu notation (see user list) I needed a >function to flip a stencil. LilyPond doesn¹t have one, so I thought it >would be worth adding the one I wrote (see below). > >Any thoughts on this before I submit t

stencil-flip function

2015-05-02 Thread Paul Morris
Hello dev list, Recently while working on Jianpu notation (see user list) I needed a function to flip a stencil. LilyPond doesn’t have one, so I thought it would be worth adding the one I wrote (see below). Any thoughts on this before I submit the code for review? (The other option would b

issue 1557: Migration from texi2html to texi2any

2015-05-02 Thread Jean-Charles Malahieude
Hi all! Texinfo evolves, and so does texi2any. What I've tried so far is: 1- build and install a new 5.9.90 texinfo 2- uninstall texi2html 3- ln -s /usr/bin/texi2any ~/ME/bin/texi2html 4- replace any occurrence of "@documentlanguage utf-8" by "@documentlanguage UTF-8" and add it where needed

Re: Part combiner: generate mark events in scheme rather than C++ (issue 228700043 by nine.fierce.ball...@gmail.com)

2015-05-02 Thread dak
https://codereview.appspot.com/228700043/diff/40001/ly/music-functions-init.ly File ly/music-functions-init.ly (right): https://codereview.appspot.com/228700043/diff/40001/ly/music-functions-init.ly#newcode1135 ly/music-functions-init.ly:1135: \context Voice = "one" \with { #one-context-settings

Re: Partcombiner documentation (Issue 4307) (issue 233110043 by philehol...@googlemail.com)

2015-05-02 Thread tdanielsmusic
https://codereview.appspot.com/233110043/diff/1/Documentation/notation/simultaneous.itely File Documentation/notation/simultaneous.itely (right): https://codereview.appspot.com/233110043/diff/1/Documentation/notation/simultaneous.itely#newcode931 Documentation/notation/simultaneous.itely:931: @n

Re: Doc: NR section 3.5.x MIDI file creation tidy up (issue 120480043 by pkx1...@gmail.com)

2015-05-02 Thread tdanielsmusic
Hi James I'm happy with this now. Thanks for being so forbearing and persistent in the face of all the criticism. So, LGTM! Trevor https://codereview.appspot.com/120480043/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.o

Re: Issue 4365: Add ly_is_smob(S) to check if S is a smob of type T. (issue 236850043 by nine.fierce.ball...@gmail.com)

2015-05-02 Thread nine . fierce . ballads
On 2015/05/02 16:11:20, Dan Eble wrote: ly_unsmob (s) which compiles iff T is the base smob type That sounds a little unclear. I mean that T owns the smob tag; it is not derived. https://codereview.appspot.com/236850043/ ___ lilypond-devel mail

Re: Issue 4365: Add ly_is_smob(S) to check if S is a smob of type T. (issue 236850043 by nine.fierce.ball...@gmail.com)

2015-05-02 Thread nine . fierce . ballads
On 2015/05/02 14:28:11, dak wrote: On 2015/05/02 12:54:18, Dan Eble wrote: > There might be some cases where ly_is_smob() is called where only > ly_is_smob() is required (though this seems unlikely). The right way to > improve performance in those cases is to ask the right question instead.

Re: gcc: error: @PYTHON_CFLAGS@: No such file or directory

2015-05-02 Thread David Kastrup
Dan Eble writes: > I’m using LilyDev 3. I just ran configure, and now I get this error > very shortly after I try to compile: > >> gcc: error: @PYTHON_CFLAGS@: No such file or directory >> make[1]: *** [out/midi.lo] Error 1 ./autogen.sh -- David Kastrup __

Re: Doc: NR section 3.5.x MIDI file creation tidy up (issue 120480043 by pkx1...@gmail.com)

2015-05-02 Thread pkx166h
On 2015/04/21 22:06:12, Trevor Daniels wrote: On 2015/04/20 16:31:52, J_lowe wrote: > > https://codereview.appspot.com/120480043/diff/240001/Documentation/notation/input.itely#newcode2732 > Documentation/notation/input.itely:2732: accent, marcato and portato. > On 2014/12/28 23:40:29, Trevor

gcc: error: @PYTHON_CFLAGS@: No such file or directory

2015-05-02 Thread Dan Eble
I’m using LilyDev 3. I just ran configure, and now I get this error very shortly after I try to compile: > gcc: error: @PYTHON_CFLAGS@: No such file or directory > make[1]: *** [out/midi.lo] Error 1 It fails in this way on this commit but not the previous: > commit 2eb22fd223cd56a2dd90eda1499d

Re: Issue 4365: Add ly_is_smob(S) to check if S is a smob of type T. (issue 236850043 by nine.fierce.ball...@gmail.com)

2015-05-02 Thread dak
On 2015/05/02 12:54:18, Dan Eble wrote: On 2015/05/02 08:25:09, dak wrote: > Nope. dynamic_cast is non-trivial operation that cannot be optimized away at > compile time for almost any use. Calling it for every instance of the rather > ubiquitous is_smob query (ubiquitous enough that I ma

Partcombiner documentation (Issue 4307) (issue 233110043 by philehol...@googlemail.com)

2015-05-02 Thread PhilEHolmes
Reviewers: Keith, Trevor Daniels, Message: Please review Description: Adds a section documenting the use of the optional argument to the partcombiner Please review this at https://codereview.appspot.com/233110043/ Affected files (+29, -0 lines): M Documentation/notation/simultaneous.itely

Re: Issue 4365: Add ly_is_smob(S) to check if S is a smob of typeT. (issue 236850043 by nine.fierce.ball...@gmail.com)

2015-05-02 Thread Phil Holmes
- Original Message - From: To: Cc: ; Sent: Saturday, May 02, 2015 1:54 PM Subject: Re: Issue 4365: Add ly_is_smob(S) to check if S is a smob of typeT. (issue 236850043 by nine.fierce.ball...@gmail.com) On 2015/05/02 08:25:09, dak wrote: Nope. dynamic_cast is non-trivial operati

Re: Issue 4365: Add ly_is_smob(S) to check if S is a smob of type T. (issue 236850043 by nine.fierce.ball...@gmail.com)

2015-05-02 Thread nine . fierce . ballads
On 2015/05/02 08:25:09, dak wrote: Nope. dynamic_cast is non-trivial operation that cannot be optimized away at compile time for almost any use. Calling it for every instance of the rather ubiquitous is_smob query (ubiquitous enough that I made it an operation separate from unsmob and ch

Re: Issue 4365: Add ly_is_smob(S) to check if S is a smob of type T. (issue 236850043 by nine.fierce.ball...@gmail.com)

2015-05-02 Thread dak
https://codereview.appspot.com/236850043/diff/1/lily/include/smobs.hh File lily/include/smobs.hh (right): https://codereview.appspot.com/236850043/diff/1/lily/include/smobs.hh#newcode235 lily/include/smobs.hh:235: template Nope. dynamic_cast is non-trivial operation that cannot be optimized aw