bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#28476: 26.0.60; Gnus: broken timezone in citation line


From: Paul Eggert
Subject: bug#28476: 26.0.60; Gnus: broken timezone in citation line
Date: Sun, 17 Sep 2017 20:56:05 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0

Thanks for the bug report.

I see two issues here. First, the change in message-citation-line-format's treatment of %Z was deliberate. The new treatment (unlike the old one) is consistent with format-time-string and this consistency simplifies the documentation and code. If backward-compatibility is a significant hassle we can re-complicate Emacs, but I'm hoping that users can change %Z to %z in their settings and move on. In the meantime I installed the first attached patch into the emacs-26 branch so that the documentation matches the new behavior. (Sorry, I missed that part of the documentation before.)

Second, (format-time-string "%Z" nil -25200) returns "+07" where it should return "-07" since -25200 denotes 7 hours west of UTC. This is due to a blunder in my commit fdb1ba144ca61185e6457f092f38f59dd9bbe6a0 dated 2016-04-12 09:19:11 -0700: the %Z code gets the sign wrong on purely-numeric time zones west of Greenwich. To fix this bug I installed the second attached patch into the emacs-26 branch. This fix should cause your example to come out as:

  On 16 September 2017 17:18 -07, Mike Kupfer wrote:

which at least gets the sign right.

Attachment: 0001-message-citation-line-format-Z-is-now-tz-name.patch
Description: Text Data

Attachment: 0002-Fix-format-time-string-Z-bug-with-negative-tz.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]