- **labels**: --> Fixed_2_19_36
- **status**: Started --> Fixed
- **Patch**: push -->
- **Comment**:
Thanks Heikki
author Heikki Tauriainen <[email protected]>
Thu, 7 Jan 2016 08:05:27 +0000 (10:05 +0200)
committer James Lowe <[email protected]>
Fri, 15 Jan 2016 19:51:33 +0000 (19:51 +0000)
commit c27954a2ec2c13e14666408982e156bea20e806a
---
** [issues:#4730] Patch: Do not output CC#7 events in MIDI on dynamic changes**
**Status:** Fixed
**Labels:** Fixed_2_19_36
**Created:** Fri Jan 08, 2016 03:54 PM UTC by H T LilyPond
**Last Updated:** Fri Jan 15, 2016 06:40 PM UTC
**Owner:** H T LilyPond
Following up on my own comment in
<https://lists.gnu.org/archive/html/lilypond-user/2015-10/msg00098.html>,
> I've observed that LilyPond actually emits CC#7 events to a generated
> MIDI file on dynamic (MIDI velocity) changes created by the
> Dynamic_performer, but these CC#7 changes seem to always just set the
> MIDI volume to 100 (in the range from 0 to 127). To me this looks like
> a possible bug (and could be just a remnant of the implementation prior
> to 2.14), but this is only my personal opinion...
I've tried to fix this defect to make the handling of MIDI CC#7 more consistent
with how the other MIDI channel controls are handled (that is, avoid enforcing
any "default" values on any controls).
(This is in preparation for a larger patch to follow for making all MIDI
controls adjustable from within LilyPond code using context properties; for
that change, it's best to first reduce any unnecessary automatic adjustment of
the MIDI channel controls.)
The patch is available for review (with further technical information) at
<https://codereview.appspot.com/277700043/>.
---
Sent from sourceforge.net because [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
