`Today` conditional date broken about neomutt HOT 4 OPEN

Looks like I tidied up the date logic a bit too much. Sorry.

Please can you give branch [devel/padding] a try.

Thanks

I've changed the rules so that:

• Plain: %d means this day (since midnight, local time)
• Number: %0d means this day (since midnight, local time)
• Number: %1d means in the last whole Day (24 hours)
• Number: %3d means in the last 3 Days (72 hours)

Now I need to create some tests and clarify the docs.

NeoMutt running in three timezones: Los Angeles, London, Tokyo.
Each has a different view of which emails arrived "today".

from neomutt.

No worries. Yep, that branch fixes the issue, thanks for the extremely quick turnaround.

If I may make a suggestion, maybe you could leave the semantics of the %Nd variable to mean "N days since midnight local time" for any number N (with the default %d == %0d), and introduce a separate %ND variable to mean "N increments of 24 hours" (probably with the default %D == %1D in this case). The two definitions of "day" are distinct and have separate uses, so it could be useful to have both available.

If you think that would be too much, the current solution fixes the bug and works for me anyway.

from neomutt.

Great!

Thanks for the suggestions, I'll have a think (about how much more that'll be :-)

from neomutt.

Thinking about it for like another 30 seconds, you already have the %H variable, so adding a second variable %D that just multiplies that by 24 is kind of pointless. So probably %Nd could just be "N days since midnight local time" and a user could specify %NH with N = 24*x to go from hours to days ago.

from neomutt.