[m-rev.] for review: lift restriction on formatting 64-bit integers
Julien Fischer
jfischer at opturion.com
Thu Jan 7 10:41:24 AEDT 2021
On Thu, 7 Jan 2021, zoltan.somogyi wrote:
> The prednames in simplify proc are missing 6s.
Fixed.
> After "NOTE: C does not provide", why is there
> only one new predicate for int64s (for base 10), while there are several for uint64s (for all possible bases)? Do the predicates for nondecimal bases already exist for int64s?
> (I cant check since i am writing this on my phone, not having access to my laptop just now.) Or are they not needed? If so. why?
They are not needed, the [oxXu] specifiers all produce an unsigned
conversion. We handle int64s for those cases by first casting to uint64
and then forwarding to the uint64 code that handles those cases. I'll
add a comment about that.
> Under the circumstances, i cant do all the checks i would want, but the rest of the diff looked good to me.
Thanks for that.
Julien.
More information about the reviews
mailing list