Code review comment for lp://staging/~smspillaz/compiz-libcompizconfig/refactor-context

Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

I can think of no good reason that ccsPrivate is void* instead of CCSContextPrivate* - that would allow the implementation code to be simpler by removing the need for casts.

And I don't see why CONTEXT_PRIV and _CCSContextPrivate/CCSContextPrivate are defined in a header.

Which is "the next pipe"?

« Back to merge proposal