* [PATCH] 12385 tst.subr.d needs fixing for NULL as a pointer
@ 2020-03-12 14:28 John Levon
2020-03-12 14:47 ` [developer] " Robert Mustacchi
0 siblings, 1 reply; 3+ messages in thread
From: John Levon @ 2020-03-12 14:28 UTC (permalink / raw)
To: developer
Please review
http://cr.illumos.org/~webrev/jlevon/
thanks
john
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [developer] [PATCH] 12385 tst.subr.d needs fixing for NULL as a pointer
2020-03-12 14:28 [PATCH] 12385 tst.subr.d needs fixing for NULL as a pointer John Levon
@ 2020-03-12 14:47 ` Robert Mustacchi
2020-03-12 15:26 ` John Levon
0 siblings, 1 reply; 3+ messages in thread
From: Robert Mustacchi @ 2020-03-12 14:47 UTC (permalink / raw)
To: illumos-developer, John Levon
On 3/12/20 7:28 AM, John Levon wrote:
>
> Please review
>
> http://cr.illumos.org/~webrev/jlevon/
Assuming http://cr.illumos.org/~webrev/jlevon/12385/ is the right url,
seems reasonable. Though I wonder if this is something the D compiler
should take care of or not.
Robert
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [developer] [PATCH] 12385 tst.subr.d needs fixing for NULL as a pointer
2020-03-12 14:47 ` [developer] " Robert Mustacchi
@ 2020-03-12 15:26 ` John Levon
0 siblings, 0 replies; 3+ messages in thread
From: John Levon @ 2020-03-12 15:26 UTC (permalink / raw)
To: Robert Mustacchi; +Cc: illumos-developer
On Thu, Mar 12, 2020 at 07:47:40AM -0700, Robert Mustacchi wrote:
> On 3/12/20 7:28 AM, John Levon wrote:
> >
> > Please review
> >
> > http://cr.illumos.org/~webrev/jlevon/
>
> Assuming http://cr.illumos.org/~webrev/jlevon/12385/ is the right url,
> seems reasonable. Though I wonder if this is something the D compiler
> should take care of or not.
Thanks. Looks like I need to re-fix my script
I don't think NULL should be treated specially IMO, but it might be
meaningful to auto-translate pointers into uintptr_t...
regards
john
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-03-12 15:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-12 14:28 [PATCH] 12385 tst.subr.d needs fixing for NULL as a pointer John Levon
2020-03-12 14:47 ` [developer] " Robert Mustacchi
2020-03-12 15:26 ` John Levon
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 NNTP newsgroup(s).