This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix build failure due to compiler warning.


From: Mark Kettenis <mark.kettenis@xs4all.nl>
Date: Wed, 23 Nov 2011 15:41:33 +0100 (CET)

>> Date: Wed, 16 Nov 2011 01:12:17 -0500 (EST)
>> From: David Miller <davem@davemloft.net>
>> 
>> On Sparc/Linux, __suseconds_t is not a type compatible with 'long' so
>> these printf statements trigger a warning and thus a build failure.
>> 
>> This mirrors the casting already being done for the same exact reason
>> in vprintf_unfiltered.
>> 
>> Ok to commit?
> 
> Not sure if you already committed this, but "long int" sounds a bit
> pedantic, so I'd prefer to see this with "long".  But either way, this
> is ok.

I did commit this, and I did use "long" as is mentioned in both my
description above and the ChangeLog entry :-)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]