Comment Actions
Sure, why not (*). 018f3d1b7bb4 was an improvement already, now you go one step further and eliminate a potentially out-of-sync variable, taking advantage of the boolean nature of the set of states.
Feel free to commit, patch LGTM. Given that we have isHidden, isVisible, isEnabled, isFullScreen etc, perhaps it would make sense to rename to isAdjusting?
(*) Let's assume the "cleanup" vs. "never touch a running system" consideration balances out in our favour here…