Code review comment for lp:~mc-return/compiz/compiz.merge-fix1101549-and-fix1101561-missing-break

Revision history for this message
MC Return (mc-return) wrote :

> Yep, you've got the right implementation.
>
> Generally a fallthough to damage handling through default: is harmless, but
> putting the break statement there will make coverity shut up, which is
> probably a good thing.

Oh, great. Thx 4 the review & the information :)

« Back to merge proposal