Re: [PATCH 3/3] reftable tests: avoid "int" overflow, use "uint64_t"
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Jan 11, 2022, 19:41 UTC
- Message-ID
- <Yd3dao5Qh5yl/cBs@nand.local>
- In-Reply-To
- <CAFQ2z_M0Fjjx8tAd_cXqk9K9q43hwienEy=UsLJi2aW8G3sEwA@mail.gmail.com>
On Tue, Jan 11, 2022 at 08:31:47PM +0100, Han-Wen Nienhuys wrote:
Show 6 quoted lines
> On Tue, Jan 11, 2022 at 8:28 PM Taylor Blau <me@ttaylorr.com> wrote: > > In any case, you're only setting the lower half of `min` high. Maybe: > > > > uint64_t min = ~0ul; > > yeah, that works.
I'm pretty sure this is OK on 32-bit systems, too, but confirmation from somebody more confident than I in this area would be welcome :).
Thanks, Taylor