Merge pull request #2456 from softrabbit/issue2318
Song: Don't remove controller just to be sure before adding it in Song::restoreControllerStates, Song::addController handles double adds. Fixes #2318.
This commit is contained in:
@@ -1198,12 +1198,6 @@ void Song::restoreControllerStates( const QDomElement & element )
|
||||
Controller * c = Controller::create( node.toElement(), this );
|
||||
Q_ASSERT( c != NULL );
|
||||
|
||||
/* For PeakController, addController() was called in
|
||||
* PeakControllerEffect::PeakControllerEffect().
|
||||
* This line removes the previously added controller for PeakController
|
||||
* without affecting the order of controllers in Controller Rack
|
||||
*/
|
||||
Engine::getSong()->removeController( c );
|
||||
addController( c );
|
||||
|
||||
node = node.nextSibling();
|
||||
|
||||
Reference in New Issue
Block a user