Code review comment for lp:~bregma/compiz/lp-1085581

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

We should probably check the other asserts in the source tree too -
we've been bitten by this change in a few areas now.

On Wed, Dec 5, 2012 at 9:14 AM, Daniel van Vugt
<email address hidden> wrote:
> WTF!? Who put important code inside asserts?
> --
> https://code.launchpad.net/~bregma/compiz/lp-1085581/+merge/137880
> Your team Compiz Maintainers is subscribed to branch lp:compiz.

--
Sam Spilsbury

« Back to merge proposal