This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] Fix cygwin32 failure introduced by [patch] windows-nat.c: Fix offset problem in signal string handling
- From: Eli Zaretskii <eliz at gnu dot org>
- To: gdb-patches at sourceware dot org
- Date: Wed, 27 Mar 2013 13:22:47 +0200
- Subject: Re: [RFA] Fix cygwin32 failure introduced by [patch] windows-nat.c: Fix offset problem in signal string handling
- References: <20130319151436 dot GB20727 at calimero dot vinschen dot de> <20130319212554 dot GE4506 at adacore dot com> <000101ce2a6b$8c855a60$a5900f20$%muller at ics-cnrs dot unistra dot fr> <83y5d9xrqt dot fsf at gnu dot org> <005501ce2ac8$ae830150$0b8903f0$%muller at ics-cnrs dot unistra dot fr> <83obe5xgq2 dot fsf at gnu dot org> <20130327111301 dot GA28863 at calimero dot vinschen dot de>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Wed, 27 Mar 2013 12:13:01 +0100
> From: Corinna Vinschen <vinschen@redhat.com>
>
> > > ../../src/gdb/windows-nat.c: In function 'handle_output_debug_string':
> > > ../../src/gdb/windows-nat.c:993:16: erreur: assignment makes pointer from integer without a cast
> >
> > Right. But IMO the casts here are dubious to begin with. strtoull
> > produces a 64-bit value; casting it to a 32-bit uintptr_t might shut
>
> This is really no problem. We know what value comes out of Cygwin.
> It's a 32 bit pointer value on 32 bit and a 64 bit value on 64 bit,
> so the cast is always right.
It may be right, but clean it isn't. I guess we will have to
disagree.
> Please stop CCing me.
It's not me, it's the "reply" function of my mailer. If you don't
want to get a copy, please set up your mail headers to cause that.