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

2009-07-14 Thread Neil Puttock
2009/7/12 Carl Sorensen c_soren...@byu.edu: 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

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 n.putt...@gmail.com wrote: 2009/7/12 Carl Sorensen c_soren...@byu.edu: 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

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

2009-07-12 Thread Neil Puttock
2009/7/12 Carl Sorensen c_soren...@byu.edu: 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

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 n.putt...@gmail.com wrote: 2009/7/12 Carl Sorensen c_soren...@byu.edu: 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,

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

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

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

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 jonlancek...@gmail.com 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