Code review comment for lp:~compiz-team/compiz/compiz.fix_1050796

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

Looks and works fine to me.

A couple of potential issues in ccsCopyValue:
  1. This is not C: 872 + if (!value)
  2. Unreachable code: 918 + return NULL;

But when and if those become a problem, the compiler will tell you.

review: Approve

« Back to merge proposal