Re: git: 4456846a1a0d - main - bin/date: Upgrade calculations
- In reply to: Warner Losh : "Re: git: 4456846a1a0d - main - bin/date: Upgrade calculations"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Tue, 04 Jul 2023 04:49:27 UTC
OK done .. and you are right .. the fix was my responsibility .. if I didn't have the time to complete it I should have left it for a while longer in my disk :-(. For the record, Dragonfly added incomplete support for the GNU -d option, but I didn't find that interesting, and I have no plans to merge that. Pedro. On Monday, July 3, 2023 at 11:37:08 PM GMT-5, Warner Losh <imp@bsdimp.com> wrote: 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