GNU bug report logs - #68969
[PATCH] Fix handling of delta values with negative month field

Previous Next

Full log

Message #11 received at 68969 <at> debbugs.gnu.org (full text, mbox):

From: Łukasz Stelmach <stlman <at> poczta.fm>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: 68969 <at> debbugs.gnu.org, Łukasz Stelmach <stlman <at> poczta.fm>
Subject: Re: bug#68969: [PATCH] Fix handling of delta values with negative
 month field
Date: Wed, 26 Feb 2025 14:12:13 +0100

[Message part 1 (text/plain, inline)]

Stefan Kangas <stefankangas <at> gmail.com> writes:

> Łukasz Stelmach <stlman <at> poczta.fm> writes:
>
>> * lisp/calendar/time-date.el (decoded-time-add): If the new
>> variable is less then zero, the year needs to be decremented
>> by quotient of new and 12 increased by one.
>> * test/lisp/calendar/time-date-tests.el (test-decoded-add):
>> Add applicable test cases.
>> ---
>>  lisp/calendar/time-date.el            |  2 +-
>>  test/lisp/calendar/time-date-tests.el | 12 ++++++++++++
>>  2 files changed, 13 insertions(+), 1 deletion(-)
>
> Thanks, and sorry for the late reply here.
>
> Could you please explain what is the issue that this patch fixes?  Is
> there a concrete use case that made you run into this?

I was playing with Gnus' message list limiting
gnus-summary-limit-to-age[1] and making my own
lksz-gnus-summary-limit-to-month before sending this patch, but honestly
I can't remember exactly. Beginning of a year may suggest that I was
moving some e-mail messages between archive folders and something wasn't
right across the year boundary. This is as close as I can guess.

I still believe, however, the patch as it is needed and makes sense.

[1] https://debbugs.gnu.org/cgi/bugreport.cgi?bug=58820
-- 
Kind regards,
Łukasz Stelmach

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.