This is the mail archive of the gdb@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 gdb]: Fix display for LLP64 target in window-nat.c

2013/2/19 Joel Brobecker <brobecker@adacore.com>:
>> You should use %s & host_address_to_string instead of %p &
>> the various casts. So the following should work:
>>
>>       DEBUG_MEM (("gdb: write target memory, %d bytes at 0x%s\n",
>>                 len, host_address_to_string (memaddr)));
>>
>> OK with that change.
>
> Actually, I was confused. Sorry. Try either core_addr_to_string_nz
> (or core_addr_to_string if you want leading zeros)...
>

Ok, I was now confused too due host_address_to_string is requiring an
pointer as input.

Just re-testing adjusted patch and will commit then.

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