From: Tom Lane <tgl@sss.pgh.pa.us>
To: Julien Rouhaud <rjuju123@gmail.com>
cc: rekgrpth@gmail.com, pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #18248: Expression is always zero
In-reply-to: <ZXrirGsvtY3PnlfB@jrouhaud>
References: <18248-4250c882e2a8b8f6@postgresql.org>
 <ZXrirGsvtY3PnlfB@jrouhaud>
Comments: In-reply-to Julien Rouhaud <rjuju123@gmail.com>
	message dated "Thu, 14 Dec 2023 12:10:36 +0100"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <2780887.1702564848.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Thu, 14 Dec 2023 09:40:48 -0500
Message-ID: <2780888.1702564848@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/2780888.1702564848%40sss.pgh.pa.us>
Precedence: bulk

Julien Rouhaud <rjuju123@gmail.com> writes:
> On Thu, Dec 14, 2023 at 05:32:49AM +0000, PG Bug reporting form wrote:
>> The value of expression 'thistimecnt >> 31 >> 31 >> 2' (in
>> src/timezone/zic.c) is zero regardless of the value of the left operand

> Note that this is code imported as-is from upstream IANA tzcode, so we w=
ouldn't
> make any change ourself.
> Looking at their git history they added it to remove some theoretical in=
t
> limits:
> https://github.com/eggert/tz/commit/cc8aec6ecb1ecc3099f173e2bca9785ae835=
9c03

Yeah, the tzcode crew have a wider range of (possibly theoretical)
target systems in mind than we do.  They probably won't think this
is a bug, and we're unlikely to diverge from whatever they do.

			regards, tom lane