* Re 2da39de "fbsd-nat: Pacify gcc with no functional changes"
@ 2023-09-25 23:39 Arsen Arsenović
2023-09-26 11:10 ` Enze Li
0 siblings, 1 reply; 3+ messages in thread
From: Arsen Arsenović @ 2023-09-25 23:39 UTC (permalink / raw)
To: Enze Li; +Cc: Gdb
[-- Attachment #1: Type: text/plain, Size: 823 bytes --]
Hi,
This patch seems to have broken building GDB on
x86_64-portbld-freebsd13.1 for me. In the following hunk...
@@ -2094,6 +2092,9 @@ fbsd_nat_target::detach (inferior *inf, int from_tty)
}
}
}
+#else
+ /* pacify gcc */
+ wptid = (void) null_ptid;
#endif
sig = 0;
break;
... in the #else branch, there's a cast which I can't discern the
purpose of. The type of wptid is the same as the type of null_ptid, and
a cast to void as an rvalue doesn't make sense. Dropping the cast
results in the build working again (and no warnings).
Did you mean for this line to be 'wptid = null_ptid'?
TIA, have a lovely night :-)
--
Arsen Arsenović
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 381 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Re 2da39de "fbsd-nat: Pacify gcc with no functional changes"
2023-09-25 23:39 Re 2da39de "fbsd-nat: Pacify gcc with no functional changes" Arsen Arsenović
@ 2023-09-26 11:10 ` Enze Li
2023-09-26 12:35 ` Arsen Arsenović
0 siblings, 1 reply; 3+ messages in thread
From: Enze Li @ 2023-09-26 11:10 UTC (permalink / raw)
To: Arsen Arsenović; +Cc: Gdb
Hi Arsen,
Thanks for your feedback, I'll fix this soon.
Thanks,
Enze
On Tue, Sep 26 2023 at 01:39:48 AM +0200, Arsen Arsenović wrote:
> Hi,
>
> This patch seems to have broken building GDB on
> x86_64-portbld-freebsd13.1 for me. In the following hunk...
>
> @@ -2094,6 +2092,9 @@ fbsd_nat_target::detach (inferior *inf, int from_tty)
> }
> }
> }
> +#else
> + /* pacify gcc */
> + wptid = (void) null_ptid;
> #endif
> sig = 0;
> break;
>
> ... in the #else branch, there's a cast which I can't discern the
> purpose of. The type of wptid is the same as the type of null_ptid, and
> a cast to void as an rvalue doesn't make sense. Dropping the cast
> results in the build working again (and no warnings).
>
> Did you mean for this line to be 'wptid = null_ptid'?
>
> TIA, have a lovely night :-)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Re 2da39de "fbsd-nat: Pacify gcc with no functional changes"
2023-09-26 11:10 ` Enze Li
@ 2023-09-26 12:35 ` Arsen Arsenović
0 siblings, 0 replies; 3+ messages in thread
From: Arsen Arsenović @ 2023-09-26 12:35 UTC (permalink / raw)
To: Enze Li; +Cc: Gdb
[-- Attachment #1: Type: text/plain, Size: 177 bytes --]
Hi Enze,
Enze Li <enze.li@hotmail.com> writes:
> Hi Arsen,
>
> Thanks for your feedback, I'll fix this soon.
Thank you!
Have a lovely day.
--
Arsen Arsenović
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 381 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-09-26 12:36 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-25 23:39 Re 2da39de "fbsd-nat: Pacify gcc with no functional changes" Arsen Arsenović
2023-09-26 11:10 ` Enze Li
2023-09-26 12:35 ` Arsen Arsenović
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).