An improvement. But...
Important:
You also need to update src/privatescreen/tests/test-privatescreen.cpp
Less important:
It doesn't handle cases where a blacklisted plugin would load on a subsequent attempt - e.g. because it got installed while compiz was running.
I also have a preference for giving better names to a many of the variables here.
PS I do agree about using find() not count() - but both are O(log(n)) in an RB-tree, not liner.
« Back to merge proposal
An improvement. But...
Important:
You also need to update src/privatescre en/tests/ test-privatescr een.cpp
Less important:
It doesn't handle cases where a blacklisted plugin would load on a subsequent attempt - e.g. because it got installed while compiz was running.
I also have a preference for giving better names to a many of the variables here.
PS I do agree about using find() not count() - but both are O(log(n)) in an RB-tree, not liner.