[quake3] -Werror patch

Jamie Wilkinson jaq at spacepants.org
Tue Sep 6 06:34:02 EDT 2005


This one time, at band camp, Ian Kumlien wrote:
>On Tue, 2005-09-06 at 19:47 +1000, Jamie Wilkinson wrote:
>> If you add -Werror to the CFLAGS, only one showstopper appears with GCC 3.3
>> on Ubuntu.
>> 
>> This patch changes the format arg to an int from a longint, given that the
>> argument is passed in as an int and the other operand is cast to int *, this
>> seems reasonable.
>
>Sorry but it's not, since now you broke it for all 64 bit machines...
>It's also kinda strange, since a long is the same size as a int on a 32
>bit machine. Anyways, Here is a fix.

Ah, shit.  Sorry.  Thanks for the fix.  I was wondering about the pointer
math, and thinking maybe %p should have been used instead.



More information about the quake3 mailing list