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

[Doug Evans <dje@google.com>]

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.