[cairo] Re: [cairo-commit] src/cairo-xcb-surface.c

Owen Taylor otaylor at redhat.com
Tue Oct 17 11:38:31 PDT 2006


On Tue, 2006-10-17 at 14:28 -0400, Behdad Esfahbod wrote:
> On Tue, 2006-10-17 at 11:11 -0400, Ian Osgood wrote:
> > 
> > +#include "cairo-clip-private.h"        /* XXX */ 
> 
> When committing to cairo please avoid comments like this :).  If there
> is something you want to remind the reader here, you really should also
> write what it is.  You are not the only one reading the code later.  In
> fact, in the long run, you definitely are not the one who needs to be
> able to understand the comment!

And in addition, when *you* read this again, you will have no idea
what you meant :-)
						- Owen




More information about the cairo mailing list