Code review comment for lp:~vanvugt/compiz/fix-1019337.2

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Yes, it is possible result != error. Because all the docs I found said that result is either Success or undefined. However 'error' tells you what the error was.

However it's a moot point since all we're testing for is Success. So there's no need to check 'error == Success' too. It's redundant but harmless.

« Back to merge proposal