Regarding versioning, I thought removing local brushes which names and values match exactly the current default 'embedded' brushes would do the trick? The others would be kept as local brushes, up…
@lichtwerk think this is more for @JulianEisel first?
Besides point raised by @HooglyBoogly, LGTM. But don't think this should go in 3.5, it's not that trivial of a change...
Not really needed indeed, but does not hurt either, will avoid setting a not-used-anymore CMake var in the mean time. ;)
@PratikPB2123 not sure how a liboverride/animation report ended up in pipeline module...
Think this is more for animation module to check first anyway, most likely specific drawing code, and/or…
Updated to use the new main invalid flag and check it in a few 'critical' places, instead of using exceptions... This will get way more verbose if we keep adding these in the future.