Re: pgsql: Avoid -Wconversion warnings when using checksum_impl.h - Mailing list pgsql-committers

From Magnus Hagander
Subject Re: pgsql: Avoid -Wconversion warnings when using checksum_impl.h
Date
Msg-id CABUevEzoWeR_q-JztqacmATBvVMUL9rtOcpM5RACubKcmhmk3Q@mail.gmail.com
Whole thread Raw
In response to Re: pgsql: Avoid -Wconversion warnings when using checksum_impl.h  (Michael Paquier <michael@paquier.xyz>)
Responses Re: pgsql: Avoid -Wconversion warnings when using checksum_impl.h
Re: pgsql: Avoid -Wconversion warnings when using checksum_impl.h
List pgsql-committers
On Thu, Mar 5, 2020 at 3:30 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Thu, Mar 05, 2020 at 12:51:50PM -0500, David Steele wrote:
> > Looks like you changed 65535 -> 65536 before commit.  I checked the original
> > patch and it has 65535.
>
> That's my fault, thanks.  I have been playing with the surroundings
> while looking at your patch.

:/

That's a pretty dangerous mistake, and moreso because we don't have a
test around it. We should really find a way to add such a test -- just
a hardcoded page calculation that should always return the same value
perhaps?


-- 
 Magnus Hagander
 Me: https://www.hagander.net/
 Work: https://www.redpill-linpro.com/



pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: pgsql: Avoid -Wconversion warnings when using checksum_impl.h
Next
From: David Steele
Date:
Subject: Re: pgsql: Avoid -Wconversion warnings when using checksum_impl.h