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] [Windows] fix format string for 64 bit var in gdbserver


On Sat, Jul 17, 2010 at 12:37 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Friday, July 16, 2010 16:10:23 Ozkan Sezer wrote:
>> For windows targets, (x86_64-w64-mingw32, i686-w64-mingw32)
>> gcc complains:
>>
>> ../../../gdb-cvs/gdb/gdbserver/server.c: In function 'handle_query':
>> ../../../gdb-cvs/gdb/gdbserver/server.c:1542: warning: unknown
>> conversion type character 'l' in format
>> ../../../gdb-cvs/gdb/gdbserver/server.c:1542: warning: too many
>> arguments for format
>> ../../../gdb-cvs/gdb/gdbserver/server.c:1566: warning: unknown
>> conversion type character 'l' in format
>> ../../../gdb-cvs/gdb/gdbserver/server.c:1566: warning: too many
>> arguments for format
>>
>> This is due to the fact that MS printf doesn't support %lld, it uses
>> its own %I64d which gcc already knows about. The attached patch
>> changes that. OK for apply?
>
> ugh, no. ?why not use a sane define like PRIx64 from inttypes.h ?
> -mike
>

I would happily do that, however that would require inttypes
module merge from gnulib to gdb/gnulib, am I wrong?

--
Ozkan


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