Merge lp://qastaging/~smspillaz/unity/unity.fix_864758 into lp://qastaging/unity
Proposed by
Sam Spilsbury
Status: | Merged |
---|---|
Merged at revision: | 1701 |
Proposed branch: | lp://qastaging/~smspillaz/unity/unity.fix_864758 |
Merge into: | lp://qastaging/unity |
Diff against target: |
73 lines (+24/-6) 2 files modified
plugins/unityshell/src/compizminimizedwindowhandler.h (+23/-5) plugins/unityshell/src/minimizedwindowhandler.h (+1/-1) |
To merge this branch: | bzr merge lp://qastaging/~smspillaz/unity/unity.fix_864758 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Penhey (community) | Approve | ||
Review via email:
|
Description of the change
Fix bug 864758
To post a comment you must log in.
Hi Sam,
This class seems weird. Ideally we should try to avoid
global scope as much as possible, which includes static lists
if at all possible.
OMG... where to start with this. In order to gain context
about what was going on, I opened the complete file. That needs
to be fixed badly. There is a template class for absolutely
no reason, apart from adding complexity where we don't need it.
It is accessing a global variable called "screen" AFAICS.
Please tell me I'm wrong.
We have one class with two responsibilities. The static members
control the static data, and the member functions control the
minimisation of the window itself. This really needs to be
broken into two, kill the static methods, have the object
owned by the UnityScreen.
Also... why are these in the compiz namespace? That is just nuts.
About a gazillion lines of complicated code can be made simpler
here.
While this branch may well fix the bug, it doesn't make me happy.