Re: git: 4456846a1a0d - main - bin/date: Upgrade calculations

From: Warner Losh <imp_at_bsdimp.com>
Date: Tue, 04 Jul 2023 04:36:53 UTC
On Mon, Jul 3, 2023, 10:07 PM Pedro Giffuni <pfg@freebsd.org> wrote:

> (Sorry for top posting)
>
> Oh yes, the analysis is fine, and it is quicker to fix than what I had in
> mind.
>
> I'll take a look at fixing it now, but due to external issues I may have
> to leave the fix for next weekend.
>

Ah great. I'll not have time to fix this... thanks for taking care of
this... sorry if i was a It grumpy... it's been a day...

Warner

Pedro.
>
>
> On Monday, July 3, 2023 at 11:00:51 PM GMT-5, Warner Losh <imp@bsdimp.com>
> wrote:
>
>
>
>
> On Mon, Jul 3, 2023, 9:34 PM Pedro Giffuni <pfg@freebsd.org> wrote:
>
> Hmm ...
>
> Dragonfly has no armv7 or i386, so they didn't get it too wrong.
> I guess the int64_t would be a quick fix or another option, which I was
> consideirng, was to look at unsigning it but taking care of the edge cases
> ... I was too lazy for that.
>
> Please go ahead and do the quick fix ;)
>
>
> What makes you say it's a quick fix? If the calculations need 64 bits,
> int64_t is the proper data type. How is that analysis wrong?
>
> Also, it's tradition that you should fix it...
>
> Warner
>