Re: [Lilypond-auto] Issue 2934 in lilypond: The bug Context.Grob.property cleanup for \override et al

[lilypond]

Comment #6 on issue 2934 by address@hidden: The bug Context.Grob.property  
cleanup for \override et al
http://code.google.com/p/lilypond/issues/detail?id=2934

Ok, I have cut the countdown something like 7 hours short on this issue.   
The reason is that it contains a somewhat tricky merge (to keep the  
mainline bisectable), is somewhat intertwined with side issue 2932, and  
contains an update-with-convert-ly run that requires careful reverts and  
can't just be rebased if staging has moved on when any change of  
documentation has been pushed in the mean time.  So I pushed this a bit  
early to avoid other stuff getting in before it and requiring the whole  
thing to be puzzled together again as I am busy tomorrow before noon.

Here are the commits involved:

* commit 54f7187cb95349fb08d359d36b0105ecaa102d15
| Author: David Kastrup <address@hidden>
| Date:   Sat Oct 27 21:08:17 2012 +0200
|
|     Change input/regression/alter-broken.ly to include tweak-like usage
|
|     Also change override-like usage for ties to \once\alterBroken as the
|     control point manipulations are unlikely to be desirable for more than
|     a single occurence.
|
*   commit f25b23eb6fbbf83489dfac39f1908ab13a75b4b9
|\  Merge: 2188505 13da8b2
| | Author: David Kastrup <address@hidden>
| | Date:   Tue Oct 30 22:57:54 2012 +0100
| |
| |     Merge symbol_list into master
| |
| |     This merge commit comprises the commits
| |
| |     13da8b2 Rerun scripts/auxiliar/update-with-convert-ly.sh
| |     516cae5 Add more extensive rules for \tweak, \override, \revert,  
\overrideProperty
| |     7b2cfcb Change \overrideProperty to take a single symbol-list as  
path
| |     ec6b407 Change \alterBroken interface to match that of other  
tweak/overrides
| |
| |     that don't have a compilable work tree and should therefore not be
| |     separate commits in the mainline.
| |
| * commit 13da8b27aabc5d5a752d00ed1e2b99ad20f0f264
| | Author: David Kastrup <address@hidden>
| | Date:   Mon Oct 29 16:25:02 2012 +0100
| |
| |     Rerun scripts/auxiliar/update-with-convert-ly.sh
| |
| * commit 516cae503bc2b55e95604b0c0f681a6ccddcfab2
| | Author: David Kastrup <address@hidden>
| | Date:   Wed Oct 10 14:31:27 2012 +0200
| |
| |     Add more extensive rules for \tweak, \override, \revert,  
\overrideProperty
| |
| * commit 7b2cfcbba64a2e41c2d49e4806e0a4904f6c07c2
| | Author: David Kastrup <address@hidden>
| | Date:   Sun Oct 28 14:10:15 2012 +0100
| |
| |     Change \overrideProperty to take a single symbol-list as path
| |
| * commit ec6b407a65c6da5a41fe2e3d250d67a20ee65e15
|/  Author: David Kastrup <address@hidden>
|   Date:   Sat Oct 27 17:21:54 2012 +0200
|
|       Change \alterBroken interface to match that of other tweak/overrides
|
* commit 2188505a75fb36893b0ec261c6bc194a201ec66e
| Author: David Kastrup <address@hidden>
| Date:   Thu Oct 25 14:42:25 2012 +0200
|
|     Implement function check-grob-path for checking/extending grob path  
specs
|
|     From the function documentation string:
|
|     Check a grob path specification @var{path}, a symbol list (or a
|     single symbol), for validity and possibly complete it.  Returns the
|     completed specification, or @code{#f} if invalid.  If optional
|     @var{parser} is given, a syntax error is raised in that case,
|     optionally using @var{location}.  If an optional keyword argument
|     @code{#:start @var{start}} is given, the parsing starts at the given
|     index in the sequence @samp{Context.Grob.property.sub-property...},
|     with the default of @samp{0} implying the full path.
|
|     If there is no valid first element of @var{path} fitting at the given
|     path location, an optionally given @code{#:default @var{default}} is
|     used as the respective element instead without checking it for
|     validity at this position.
|
|     The resulting path after possibly prepending @var{default} can be
|     constrained in length by optional arguments @code{#:min @var{min}} and
|     @code{#:max @var{max}}, defaulting to @samp{1} and unlimited,
|     respectively.
|