[PATCH] unifdef -B -DRENDER to always include RENDER code

Keith Packard keithp at keithp.com
Mon Apr 19 15:54:08 PDT 2010


On Mon, 19 Apr 2010 14:27:11 -0700, Alan Coopersmith <alan.coopersmith at oracle.com> wrote:

> Not really part of the unifdef patch, but a second patch to do s/of/off/ in that
> message would be good.

Yeah, as you can imagine, any patch that changes as much as the RENDER
stuff should be entirely mechanical.

> Also, you seem to have somehow pulled in some non-RENDER related changes (perhaps
> because it realized the && 0 made these always-dead already?):

Hrm. unifdef has the '-k' option (which I didn't use) that is supposed
to leave #if lines that have constant expressions, and also '-K' which
leaves expressions with && and || undefined; looks like I need to add -K
to leave this particular expression alone.

> > -#if CYGMULTIWINDOW_DEBUG && 0
> > -  winDebug ("winMWExtWMUpdateRegion (%08x)\n", pRLWinPriv);
> > -#endif

I looked over the patch but clearly missed this piece.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100419/0e025c11/attachment-0001.pgp>


More information about the xorg-devel mailing list