Code review comment for lp:~vanvugt/compiz/blacklist

Revision history for this message
Michael Terry (mterry) wrote :

Will it always be obvious what "blacklisted" means? Right now it seems it is specific to one feature. We might get a second feature in the future that needs a separate blacklist list. Just saying that maybe the function name should be something like blacklisted_for_unredirect(). But I don't feel strongly about it.

« Back to merge proposal