Merge lp://staging/~vanvugt/compiz/fix-1019337.2 into lp://staging/compiz/0.9.8
Proposed by
Daniel van Vugt
Status: | Merged |
---|---|
Approved by: | Łukasz Zemczak |
Approved revision: | 3286 |
Merged at revision: | 3286 |
Proposed branch: | lp://staging/~vanvugt/compiz/fix-1019337.2 |
Merge into: | lp://staging/compiz/0.9.8 |
Diff against target: |
17 lines (+3/-0) 1 file modified
gtk/window-decorator/wnck.c (+3/-0) |
To merge this branch: | bzr merge lp://staging/~vanvugt/compiz/fix-1019337.2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Łukasz Zemczak (community) | Approve | ||
jenkins (community) | continuous-integration | Approve | |
Review via email:
|
Commit message
Fix crash LP: #1019337 properly this time.
The original fix did not handle the case where:
(win != NULL && xid != None && !valid_window(xid))
so would still exit with an X error.
Now we trap and handle such errors gracefully, instead of hitting the
default handler which kills gtk-window-
Description of the change
Fix crash LP: #1019337 properly this time.
The original fix did not handle the case where:
(win != NULL && xid != None && !valid_window(xid))
so would still exit with an X error.
Now we trap and handle such errors gracefully, instead of hitting the
default handler which kills gtk-window-
UNBLOCK
To post a comment you must log in.
PASSED: Continuous integration, rev:3285 jenkins. qa.ubuntu. com/job/ compiz- ci/14/ jenkins. qa.ubuntu. com/job/ compiz- ci/./distributi on=quantal, flavor= amd64/14/ console jenkins. qa.ubuntu. com/job/ compiz- ci/./distributi on=quantal, flavor= i386/14/ console
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://