[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4745 Add Staff.midiCC context property, refactor handling of MIDI control changes

[Auto mailings of changes to Lily Issues]

From the review discussion so far (which has turned more into a design discussion about the "proper" interface for this enhancement feature), I've concluded that pushing this patch without a redesign of its interface is not going to be acceptable. Can this patch be put on hold for now until the design questions have been resolved (and until there's a new patchset ready to be reviewed)?

---

[issues:#4745] Add Staff.midiCC context property, refactor handling of MIDI control changes

Status: Started
Created: Sat Jan 16, 2016 06:01 PM UTC by H T LilyPond
Last Updated: Sat Jan 30, 2016 12:12 PM UTC
Owner: H T LilyPond

Add Staff.midiCC context property, refactor handling of MIDI control changes

Setting the Staff.midiCC context property to a list of (control number,
control value) pairs, where every control number and control value must
be an integer between the inclusive range from 0 to 127, will send the
corresponding raw control changes to the MIDI output on the MIDI channel
associated with the current context. For example,
\set Staff.midiCC = #'((7 . 100)) will set the MIDI volume control (#7)
to 100.

Most of the logic for handling MIDI control value initialization from
context properties (in Staff_performer::new_audio_staff),
changes (in
Midi_control_function_performer::announce_function_value_change), and
value conversion for output (in
Midi_control_function_value_change::to_string), was moved into the new
Midi_control_change_announcer class. All MIDI control changes are now
encoded using {Audio,Midi}_control_change items. This change makes the
old {Audio,Midi}_control_function_value_change classes obsolete.

http://codereview.appspot.com/284280043

---

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.

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140

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