[m-rev.] for review: Remove MR_INT_LEAST64_LENGTH_MODIFIER.
Peter Wang
novalazy at gmail.com
Wed Dec 20 15:33:47 AEDT 2023
On Wed, 20 Dec 2023 14:43:44 +1100 Julien Fischer <jfischer at opturion.com> wrote:
>
> > diff --git a/runtime/mercury_types.h b/runtime/mercury_types.h
> > index 11085e550..53e35f56c 100644
> > --- a/runtime/mercury_types.h
> > +++ b/runtime/mercury_types.h
> > @@ -44,6 +44,7 @@
> > #endif
> >
> > // This section defines types similar to C9X's <stdint.h> header.
> > +// XXX The following is out of date.
> > // We do not use <stdint.h>, or the <inttypes.h> or <sys/types.h> files
> > // that substitute for it on some systems because (a) some such files
> > // do not define the types we need, and (b) some such files include
>
> Reason (a) is very much out of date; reason (b) (which is that
> the headers may define inline functions) is presuambly still
> a possible issue.
I'll update the comment along those lines then.
Peter
More information about the reviews
mailing list