[m-rev.] for post-commit review: group *int_least* types together

Julien Fischer jfischer at opturion.com
Wed Feb 21 15:18:36 AEDT 2018



On Wed, 21 Feb 2018, Zoltan Somogyi wrote:

>
>
> On Tue, 20 Feb 2018 05:00:55 -0500 (EST), Julien Fischer <jfischer at opturion.com> wrote:
>> I suggest making the test for the uint_least32 case something like:
>>
>>       0 =< N,
>>       ( int.bits_per_int = 32
>>       ; int64.from_int(N) < 4294967295_i64
>>       )
>
> Would the attached diff be ok? It also consistently uses
> symmetric checks, [] instead of [), partly for style,
> and partly to avoid referring to int32_max + 1.

You forgot to attach the diff.

Julien.


More information about the reviews mailing list