Skip to content

Commit

Permalink
Consolidated EffectChain and EffectChainSlot
Browse files Browse the repository at this point in the history
- EffectChain and EffectChainSlot had no clear separation of responsibility
- Had to keep the duplicated state in both the classes in sync and thus making it more susceptible to bugs

  NOTE: XML saving/loading of effects is broken and will be re-implemented in the new architecture
  • Loading branch information
kshitij98 committed Oct 18, 2018
1 parent c9ab901 commit 052fd74
Show file tree
Hide file tree
Showing 21 changed files with 544 additions and 1,196 deletions.
1 change: 0 additions & 1 deletion build/depends.py
Expand Up @@ -693,7 +693,6 @@ def sources(self, build):
"effects/effectmanifest.cpp",
"effects/effectmanifestparameter.cpp",

"effects/effectchain.cpp",
"effects/effect.cpp",
"effects/effectparameter.cpp",

Expand Down
373 changes: 0 additions & 373 deletions src/effects/effectchain.cpp

This file was deleted.

0 comments on commit 052fd74

Please sign in to comment.