On 4/07/10 15:21 , Jesse Adkins wrote:
Yes, various people have been slowly cleaning them out as we touch
libraries/apps that have them, but there's been no organized effort
to methodically purge them.
Sounds like something I can do.
I'd much rather see you spend time on something useful.
CVS tags don't hurt anyone and if you're having a day where your brain
just doesn't want to get into gear (although or possibly because you've
drowned it in coffee :) then those patches are fine.
Any other day I'd rather you use your spare cycles for technical
improvement. It benefits you (you learn the code, you get better at
programming) and everyone else (better technical product). Don't
undersell yourself, I'm sure you can do better than just search/replace.
Those have
different meanings and semantics, so changing them is not necessarily
the correct thing to do. Do you have specific
examples?
None,
actually. I've just seen '#if x' instead of '#ifdef x' in a few places.
It's probably not a good idea to get into, especially since I'm still
new to X.
many of these are for conditional compiles. we've got plenty of
#if USE_FEATURE
blah();
#endif
if USE_FEATURE isn't defined, then that part isn't compiled and the
library/server/driver builds without $FEATURE. They're not the same as
#ifdef, though they can sometimes used interchangebly - provided you
know when.
Cheers,
Peter
----- Original Message ----
From: Alan Coopersmith<[email protected]>
To: Jesse Adkins<[email protected]>
Cc: Xorg Devel<[email protected]>
Sent: Sat, July 3, 2010 9:16:51 PM
Subject: Re: Janitor ideas
Jesse Adkins wrote:
I've noticed a few things while poking around Xorg lately. Would patches to fix
the following things be of use (and where should I send them?)
Patches should be sent to xorg-devel with Signed-off-by/etc. as described on:
http://www.x.org/wiki/Development/Documentation/SubmittingPatches
* Lots of X libraries still use Xmalloc and others, instead of the regular
stdlib functions (despite the defines in Xserver now referencing the stdlib
functions).
Unlike the X server, we can't break client library ABI, so I don't think
removing Xalloc/XFree in them is an option. (This problem goes away for
clients that port to xcb, but we'll probably be stuck maintaining libX11
for at least two or three more decades.)
* I still see some xorgcvs and XFree86 cvs tags on various files in different
libraries.
Yes, various people have been slowly cleaning them out as we touch
libraries/apps that have them, but there's been no organized effort
to methodically purge them.
* Changing constructs like '#if x' to '#ifdef x' or 'if !x' to '#ifndef x'
Those have different meanings and semantics, so changing them is not necessarily
the correct thing to do. Do you have specific examples?
We should certainly add some of these to:
http://www.x.org/wiki/Development/Janitor
http://www.x.org/wiki/ToDo
(Even just performing the Janitorial task of combining both lists into one in
the wiki would be a good start. 8-) )
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
