davidedmundson requested changes to this revision. davidedmundson added a comment. This revision now requires changes to proceed. | View Revision
Concept is good.
It's weird to be caching in a public getter.
It opens us up for problems
If some client code (for whatever reason) did
connect(scope, inheritChanged, []() {scope->colorGroup());
then our signals in checkColorGroupChanged won't get emitted.
Can we move this member var into checkColorGroupChanged? Means we can get rid of the mutable too