Wall plugin: Redundant if (screen->otherGrabExist ("wall", 0)) check in WallScreen::initiateFlip
Bug #1160878 reported by
MC Return
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Fix Released
|
Low
|
MC Return | ||
compiz (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
In bool WallScreen:
if (screen-
return false;
then we check:
if (state & CompAction:
{
if (!optionGetEdge
return false;
if (screen-
return false;
}
- so this double-check is redundant and should be removed.
Related branches
lp:~mc-return/compiz/compiz.merge-fix1160878-wall-redundant-condition-check
- PS Jenkins bot (community): Approve (continuous-integration)
- Sam Spilsbury: Approve
-
Diff: 13 lines (+0/-3)1 file modifiedplugins/wall/src/wall.cpp (+0/-3)
Changed in compiz: | |
assignee: | nobody → MC Return (mc-return) |
Changed in compiz: | |
status: | New → In Progress |
Changed in compiz: | |
importance: | Undecided → Low |
Changed in compiz: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fix committed into lp:compiz at revision None, scheduled for release in compiz, milestone 0.9.9.2