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); I keep wondering whether this assignment is intended to be kept identical to the way "volume" is initialized above in the "first time only" case (before this patch, the behavior would be the same, but now it's different), and 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... https://codereview.appspot.com/308890043/ _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
