Merge lp://staging/~vanvugt/compiz/fix-1002715-1002721 into lp://staging/compiz/0.9.8

Proposed by Daniel van Vugt
Status: Merged
Merged at revision: 3209
Proposed branch: lp://staging/~vanvugt/compiz/fix-1002715-1002721
Merge into: lp://staging/compiz/0.9.8
Diff against target: 97 lines (+36/-9)
3 files modified
src/privatescreen.h (+2/-2)
src/privatescreen/tests/test-privatescreen.cpp (+31/-0)
src/screen.cpp (+3/-7)
To merge this branch: bzr merge lp://staging/~vanvugt/compiz/fix-1002715-1002721
Reviewer Review Type Date Requested Status
Sam Spilsbury Approve
Review via email: mp+106936@code.staging.launchpad.net

Description of the change

Don't drop plugins from the list to try and load before you've even tried to
load them. Doing so makes missing plugins silently ignored instead of an
error message (LP: #1002715). It also means valid plugins in more unusual, but
real locations in LD_LIBRARY_PATH will never get loaded (LP: #1002721).

To post a comment you must log in.
Revision history for this message
Sam Spilsbury (smspillaz) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches