|
From: | nine . fierce . ballads |
Subject: | Re: Issue 4936: look up "mf" for default initial volume (issue 308890043 by address@hidden) |
Date: | Thu, 04 Aug 2016 16:07:59 -0700 |
https://codereview.appspot.com/308890043/diff/1/lily/dynamic-performer.cc File lily/dynamic-performer.cc (right): https://codereview.appspot.com/308890043/diff/1/lily/dynamic-performer.cc#newcode412 lily/dynamic-performer.cc:412: volume = equalize_volume (Audio_span_dynamic::DEFAULT_VOLUME); On 2016/08/04 21:23:25, ht wrote:
[...] why the "first time" case needs to be handled separately since the control
flow will end
up in the if block starting at line 405 even in that case...
"volume" is passed as a parameter to two functions between the "first time" block and here. https://codereview.appspot.com/308890043/
[Prev in Thread] | Current Thread | [Next in Thread] |