Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2300)

Issue 7684043: Make midiMaximumVolume take effect without initial dynamic (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 1 month ago by pkx166h
Modified:
7 years, 1 month ago
Reviewers:
CC:
lilypond-devel_gnu.org, arrowyard
Visibility:
Public.

Description

Make midiMaximumVolume take effect without initial dynamic My idea is to ensure that the volume is *always* initialised in Dynamic_performer::process_music. The code to do that is mostly copied from what happens if there is an actual script event. The constant 0.71 was obtained by calculating backwards from the default velocity value (0x5a) in the Midi_note constructor, midi-item.cc line 179.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -0 lines) Patch
M lily/dynamic-performer.cc View 1 chunk +12 lines, -0 lines 0 comments Download

Messages

Total messages: 1
pkx166h
7 years, 1 month ago (2017-03-19 19:16:24 UTC) #1
author    Anders Pilegaard <arrowyard@gmail.com>   
Fri, 22 Feb 2013 08:32:18 +0000 (09:32 +0100)
committer    James Lowe <pkx166h@gmail.com>   
Sun, 17 Mar 2013 12:48:18 +0000 (12:48 +0000)
commit    [ra970166b725ea78a1b3866e0a0bd208155193222]
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b