Re: [PATCH 03/15] Make gdbserver CORE_ADDR unsigned

Doug Evans
2014-07-11T19:30:44Z

Gary Benson writes:
> gdbserver defines CORE_ADDR to be signed. This seems erroneous to
> me; and furthermore likely to cause problems in common/, as it is
> different from gdb's definition.
>
> gdb/gdbserver/
> 2014-07-09 Tom Tromey <tromey@redhat.com>
>
> * defs.h (CORE_ADDR): Now unsigned.
> ---
> gdb/gdbserver/ChangeLog | 4 ++++
> gdb/gdbserver/server.h | 2 +-
> 2 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/gdb/gdbserver/server.h b/gdb/gdbserver/server.h
> index 6eb1a16..2d55513 100644
> --- a/gdb/gdbserver/server.h
> +++ b/gdb/gdbserver/server.h
> @@ -87,7 +87,7 @@ typedef unsigned char gdb_byte;
>
> /* FIXME: This should probably be autoconf'd for. It's an integer type at
> least the size of a (void *). */
> -typedef long long CORE_ADDR;
> +typedef unsigned long long CORE_ADDR;
>
> typedef long long LONGEST;
> typedef unsigned long long ULONGEST;
> --
> 1.7.1
>

LGTM, modulo file name spelled wrong in changelog entry.