Re: Doc: NR 4.4.1: Rewrite. (issue2642043)

2010-11-04 Thread markpolesky
Here's the next patch set for the Vertical spacing inside a system doc stuff. I tried to address everything you guys mentioned; let me know what you think. One significant thing I did here was to remove the entire @subsection called Vertical spacing between systems, since I don't think it was

Re: Add tab-tie-follow-engraver (issue2723043)

2010-11-04 Thread Marc Hohl
Am 03.11.2010 21:22, schrieb Neil Puttock: On 3 November 2010 19:33, Carl Sorensenc_soren...@byu.edu wrote: But the tie callback *should* make the notehead transparent if there's no slur or gliss (or bend, in the future). In the absence of slur, gliss, or split tie the notehead is

Re: Add tab-tie-follow-engraver (issue2723043)

2010-11-04 Thread Marc Hohl
Am 03.11.2010 20:33, schrieb Carl Sorensen: On 11/3/10 10:15 AM, Marc Hohlm...@hohlart.de wrote: Am 03.11.2010 15:10, schrieb Carl Sorensen: On 11/3/10 6:50 AM, Marc Hohlm...@hohlart.de wrote: Am 02.11.2010 04:04, schrieb carl.d.soren...@gmail.com:

Re: Doc: NR 4.4.1: Rewrite. (issue2642043)

2010-11-04 Thread v . villenave
Hi Mark, It's the first time I read your NR4 patches, so I may certainly have missed a few things. Your patch looks good, but I was a bit surprised by the overall writing style of NR4. This chapter looks odd to me, and somehow not on par NR1 and 2. It's obviously a whole chapter that hasn't

Re: Tied harmonics in tablature - tracker issue

2010-11-04 Thread Marc Hohl
Am 04.11.2010 00:03, schrieb Carl Sorensen: On 11/3/10 9:34 AM, Marc Hohlm...@hohlart.de wrote: Am 03.11.2010 15:11, schrieb Carl Sorensen: On 11/3/10 7:03 AM, Marc Hohlm...@hohlart.de wrote: Hello list, hello Valentin (I think you are the master of the tracker),

Re: Doc: NR 4.4.1: Rewrite. (issue2642043)

2010-11-04 Thread tdanielsmusic
I've limited my comments to just one :) I think we need to push this now and move on - we still have to change all the names, and a further review of the wording after that might suggest a few more tweaks. I'm sure more clarification will be necessary after users try to understand and use this,

Re: renaming vertical spacing inside systems props

2010-11-04 Thread Trevor Daniels
Valentin Villenave wrote Thursday, November 04, 2010 12:10 AM On Thu, Nov 4, 2010 at 12:58 AM, Trevor Daniels t.dani...@treda.co.uk wrote: Renaming proposals, round 3: CURRENT NAME PROPOSED NAMETD's PREFERENCE ----

Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044)

2010-11-04 Thread ianhulin44
Comments actioned and tested with LilyPond using Guile V1.8.7 and V1.9.13. Ian http://codereview.appspot.com/2313044/diff/1/scm/ly-syntax-constructors.scm File scm/ly-syntax-constructors.scm (left): http://codereview.appspot.com/2313044/diff/1/scm/ly-syntax-constructors.scm#oldcode20

Re: request for help for research from Guatemala

2010-11-04 Thread James
Hello On 03/11/2010 20:35, Han-Wen Nienhuys wrote: When faced with silly questions like these, I just ignore them. It costs me even less of my time than answering RTFM. And it probably took you as much time to read the thread, process it and then type an email telling us you just ignore

Re: request for help for research from Guatemala

2010-11-04 Thread James
On 03/11/2010 21:20, Han-Wen Nienhuys wrote: My reply was really aimed at other lilypond developers -- I wanted to discourage developers from taking it seriously and wasting half an hour helpfully describing stuff in an email to such a student. I think it should up to those developers to

Re: Doc: NR 4.4.1: Rewrite. (issue2642043)

2010-11-04 Thread Carl . D . Sorensen
LGTM. Thanks, Mark. Carl http://codereview.appspot.com/2642043/ ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Doc: NR 4.4.1: Rewrite. (issue2642043)

2010-11-04 Thread percival . music . ca
http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely File Documentation/notation/spacing.itely (right): http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely#newcode256 Documentation/notation/spacing.itely:256: a system is the

follow-up to report 22

2010-11-04 Thread Graham Percival
This email is a follow-up to comments in: http://news.lilynet.net/The-LilyPond-Report-22 I spent 5 minutes trying to post my latest reply, but the forum software continually complained that I had used wrong words, tags, or symbols, and thus it rejected my comment as spam. I am posting my reply

Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044)

2010-11-04 Thread pnorcks
Hi Ian, I found a rebasing issue that should be sorted out, as explained in my comment below. Also, I think the subject line of this patch can be improved, since we're no longer removing `define-ly-syntax', just revising it. Thanks, Patrick

Re: lilypond 2.13.38

2010-11-04 Thread Patrick McCarty
On Sun, Oct 31, 2010 at 7:55 AM, Graham Percival gra...@percival-music.ca wrote: Bug squad: Phil is busy with the opening of a musical, so it would be nice if somebody else could check the regression test comparisons for both 2.13.37 and 2.13.38.  It would be a shame if some horrible bug was

Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044)

2010-11-04 Thread ianhulin44
Patchset 4 uploaded partial restored to status as after T372 fix. http://codereview.appspot.com/2313044/ ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: follow-up to report 22

2010-11-04 Thread Valentin Villenave
On Thu, Nov 4, 2010 at 6:16 PM, Graham Percival gra...@percival-music.ca wrote: I spent 5 minutes trying to post my latest reply, but the forum software continually complained that I had used wrong words, tags, or symbols, and thus it rejected my comment as spam.  I am posting my reply here

Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044)

2010-11-04 Thread v . villenave
Greetings Ian, your patch looks very clever to me; just a possible indentation issue (I'm not sure what our policy is, but IIRC it used to be whatever Emacs does). BTW: This is totally unrelated to your patch, but I was recently wondering about this TODO in music-functions-init.ly, l32: %% TODO:

Re: follow-up to report 22

2010-11-04 Thread Reinhold Kainhofer
Am Donnerstag, 4. November 2010, um 21:56:24 schrieb Valentin Villenave: 2) Agree that an OSS project can, in theory, have a private mailing list. And apologize. *sigh* I do apologize if you felt offended (which you obviously do). However, I can assure you that my goal is not to make you

Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044)

2010-11-04 Thread ianhulin44
http://codereview.appspot.com/2313044/diff/15001/scm/ly-syntax-constructors.scm File scm/ly-syntax-constructors.scm (right): http://codereview.appspot.com/2313044/diff/15001/scm/ly-syntax-constructors.scm#newcode20 scm/ly-syntax-constructors.scm:20: (defmacro define-ly-syntax (args . body) On

Re: T1249 - Remove (define define-ly-syntax define-public). (issue2313044)

2010-11-04 Thread pnorcks
Hi Ian, Just about there... Three lines with whitespace issues, and then everything should be ready to go. Thanks, Patrick http://codereview.appspot.com/2313044/diff/20001/scm/ly-syntax-constructors.scm File scm/ly-syntax-constructors.scm (right):

Re: Doc: NR 4.4.1: Rewrite. (issue2642043)

2010-11-04 Thread k-ohara5a5a
Mark, Too long, but hard to say what will be useful without getting away for a while. Looks good to me, whether you either take or leave my suggestions. http://codereview.appspot.com/2642043/diff/14001/Documentation/notation/spacing.itely File Documentation/notation/spacing.itely (right):