Re: PATCH: Consolidate autobeaming to one property that controls it

2009-07-14 Thread Carl Sorensen
On 7/14/09 4:08 PM, "Neil Puttock" wrote: > 2009/7/12 Carl Sorensen : > >> I don't know.  I didn't write displayLilyMusic; that was Nicolas IIRC.  I do >> know that \time 3/4 executes those functions.  It's really easy to go >> forward in the parse tree (i.e. from \time 3/4 to \set Timing

Re: PATCH: Consolidate autobeaming to one property that controls it

2009-07-14 Thread Neil Puttock
2009/7/12 Carl Sorensen : > I don't know.  I didn't write displayLilyMusic; that was Nicolas IIRC.  I do > know that \time 3/4 executes those functions.  It's really easy to go > forward in the parse tree (i.e. from \time 3/4 to \set Timing), but I > don't know any way to reliably go in revers

Re: PATCH: Consolidate autobeaming to one property that controls it

2009-07-12 Thread Carl Sorensen
On 7/12/09 9:43 AM, "Neil Puttock" wrote: > 2009/7/12 Carl Sorensen : > >> I've created a second patch set with the whitespace removed (along with >> automatic removal of whitespace from the lines of the files that weren't >> part of the patch). > > Brilliant work, Carl! Thanks for the comp

Re: PATCH: Consolidate autobeaming to one property that controls it

2009-07-12 Thread Neil Puttock
2009/7/12 Carl Sorensen : > I've created a second patch set with the whitespace removed (along with > automatic removal of whitespace from the lines of the files that weren't > part of the patch). Brilliant work, Carl! I'll comment on individual issues in Rietveld shortly. I notice you've chang

Re: PATCH: Consolidate autobeaming to one property that controls it

2009-07-11 Thread Carl Sorensen
On 7/11/09 6:07 PM, "Jonathan Kulp" wrote: > Carl Sorensen wrote: >> At *long* last I have completed my changes to the autobeaming rules. >> >> The rietveld patch set is available at: >> >> http://codereview.appspot.com/88155 >> >> >> The objective is to get all information that affects au

Re: PATCH: Consolidate autobeaming to one property that controls it

2009-07-11 Thread Jonathan Kulp
Carl Sorensen wrote: At *long* last I have completed my changes to the autobeaming rules. The rietveld patch set is available at: http://codereview.appspot.com/88155 The objective is to get all information that affects autobeaming in a single place, so a single override will adjust the autobe

Re: PATCH: Consolidate autobeaming to one property that controls it

2009-07-11 Thread Jonathan Kulp
Carl Sorensen wrote: At *long* last I have completed my changes to the autobeaming rules. The rietveld patch set is available at: http://codereview.appspot.com/88155 The objective is to get all information that affects autobeaming in a single place, so a single override will adjust the autobe

PATCH: Consolidate autobeaming to one property that controls it

2009-07-11 Thread Carl Sorensen
At *long* last I have completed my changes to the autobeaming rules. The rietveld patch set is available at: http://codereview.appspot.com/88155 The objective is to get all information that affects autobeaming in a single place, so a single override will adjust the autobeam settings. Autobeam