[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4911 Don't treat context modification identifiers special

[Auto mailings of changes to Lily Issues]

• Description has changed:

Diff:

• Needs: -->
  
• Patch: new --> review
• Comment:

I'm getting programming errors but I know these types of errors can happen because of the way patches are tested (or something like that), the errors are these repeated on a number of reg tests:

--snip--

input/regression/midi/rest-dynamic-midi.log

@@ -15,6 +15,6 @@
Writing ./rest-dynamic-midi-systems.tex...
Writing ./rest-dynamic-midi-systems.count...
Writing timing to rest-dynamic-midi.profile...
-programming error: Parsed object should be dead: #<Prob: Music="" C++:="" Music((value="" .="" #<Mom="" 15="">) (symbol . tempoWholesPerMinute))((display-methods #<procedure #f="" (expr)="">) (name . PropertySet) (types layout-instruction-event) (iterator-ctor . #<primitive-procedure ly:property-iterator::constructor="">) (untransposable . #t)) >
+programming error: Parsed object should be dead: #<Prob: Music="" C++:="" Music((origin="" .="" #<location="" (position="" unknown)="">) (value . #<Mom 15="">) (symbol . tempoWholesPerMinute))((display-methods #<procedure #f="" (expr)="">) (name . PropertySet) (types layout-instruction-event) (iterator-ctor . #<primitive-procedure ly:property-iterator::constructor="">) (untransposable . #t)) >

continuing, cross fingers

input/regression/midi/rest-dynamic-midi.log
1.414214

input/regression/midi/rest-midi.log

@@ -15,6 +15,6 @@
Writing ./rest-midi-systems.tex...
Writing ./rest-midi-systems.count...
Writing timing to rest-midi.profile...
-programming error: Parsed object should be dead: #<Prob: Music="" C++:="" Music((value="" .="" #<Mom="" 15="">) (symbol . tempoWholesPerMinute))((display-methods #<procedure #f="" (expr)="">) (name . PropertySet) (types layout-instruction-event) (iterator-ctor . #<primitive-procedure ly:property-iterator::constructor="">) (untransposable . #t)) >
+programming error: Parsed object should be dead: #<Prob: Music="" C++:="" Music((origin="" .="" #<location="" out-test="" rest-midi.ly:46:3="">))((display-methods #<procedure #f="" (check)="">) (name . BarCheck) (types bar-check) (iterator-ctor . #<primitive-procedure ly:bar-check-iterator::constructor="">)) >

continuing, cross fingers

input/regression/midi/rest-midi.log

--snip--

If these are real errors can you set the incident back to needs_work?

Otherwise passes make, make check and a full make doc.

---

[issues:#4911] Don't treat context modification identifiers special

Status: Started
Created: Thu Jun 30, 2016 06:40 PM UTC by David Kastrup
Last Updated: Thu Jun 30, 2016 06:40 PM UTC
Owner: David Kastrup

Don't treat context modification identifiers special

This requires using \with before context modifications like
\RemoveEmptyStaves in order to make the syntax get along
with fewer special cases and exceptions.

Also contains commits:

Run scripts/auxiliar/update-with-convert-ly.sh

convert-ly rule to put \with before context mods

\with #unspecified should be equivalent to \with { }

http://codereview.appspot.com/299460043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

------------------------------------------------------------------------------
Attend Shape: An AT&T Tech Expo July 15-16. Meet us at AT&T Park in San
Francisco, CA to explore cutting-edge tech and listen to tech luminaries
present their vision of the future. This family event has something for
everyone, including kids. Get more information and register today.
http://sdm.link/attshape

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto