[PATCH all of 265] ANSI-fied libX11

Magnus Kessler Magnus.Kessler at gmx.net
Mon Apr 30 00:31:07 PDT 2007


Hi all,

sending 265 individual patches probably isn't the smartest idea, as Alan 
Hourihane and others pointed out. I have created bug 10816 [0] for this 
issue and attached the complete patch there. If anyone want's to have it 
directly the full patch is also attached to this message.

When reviewing you will find that for most files the changes made are really 
straight forward and boring. There are some, however, such as in #19 
(ChkIfEv.c) in the series posted so far where the order of variable 
declarations did not match the order of arguments. Please check the order 
is still correct.

I made the decision early on not to do anything about the register keyword 
as I'm not sure that removing it will not change the ABI on all systems. If 
removing the register keyword is the Right Thing, a follow up patch set 
could solve this issue.

Sorry for the noise on the list.

Best regards,

Magnus

[0] https://bugs.freedesktop.org/show_bug.cgi?id=10816
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libX11-cleanup.patch
Type: text/x-diff
Size: 199450 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg/attachments/20070430/c333caed/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.x.org/archives/xorg/attachments/20070430/c333caed/attachment.pgp>


More information about the xorg mailing list