GNU bug report logs - #39001
parse-time-string fails to parse Info node's example

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 39001 in the body.
You can then email your comments to 39001 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: mark skilbeck <markskilbeck <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: parse-time-string fails to parse Info node's example
Date: Mon, 6 Jan 2020 22:01:00 -0800

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

The info node (elisp) Time Parsing suggests that the string
"1998-09-12T12:21:54-0200" should be of a format that `parse-time-string`
can successfully parse. However, trying to do so produces no useful info
from parsing (it returns a list of NILs).

GNU Emacs 26.3 (build 1, x86_64-apple-darwin18.5.0, Carbon Version 158
AppKit 1671.4) of 2019-08-29

[Message part 2 (text/html, inline)]

Message #10 received at 39001-done <at> debbugs.gnu.org (full text, mbox):

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: mark skilbeck <markskilbeck <at> gmail.com>
Cc: 39001-done <at> debbugs.gnu.org
Subject: Re: parse-time-string fails to parse Info node's example
Date: Wed, 15 Jan 2020 17:45:02 -0800

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

Thanks for the bug report. I fixed the code so that it matches the 
documentation by installing the attached patches into Emacs master. The 
first patch is the fix, the second is merely refactoring.

[0001-parse-time-string-now-parses-ISO-8601-format-strings.patch (text/x-patch, attachment)]

[0002-Refactor-parse-time-string.patch (text/x-patch, attachment)]

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

From: Stefan Kangas <stefan <at> marxist.se>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 39001 <at> debbugs.gnu.org, mark skilbeck <markskilbeck <at> gmail.com>
Subject: Re: bug#39001: closed (Re: parse-time-string fails to parse Info
 node's example)
Date: Thu, 16 Jan 2020 03:14:42 +0100

Paul Eggert <eggert <at> cs.ucla.edu> writes:

> Thanks for the bug report. I fixed the code so that it matches the documentation
> by installing the attached patches into Emacs master. The first patch is the
> fix, the second is merely refactoring.

There's a test suite in
test/lisp/calendar/parse-time-tests.el

Perhaps we should add a test or two there?

Best regards,
Stefan Kangas

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Stefan Kangas <stefan <at> marxist.se>
Cc: 39001 <at> debbugs.gnu.org, mark skilbeck <markskilbeck <at> gmail.com>
Subject: Re: parse-time-string fails to parse Info node's example
Date: Wed, 15 Jan 2020 18:20:17 -0800

On 1/15/20 6:14 PM, Stefan Kangas wrote:
> Perhaps we should add a test or two there?

Yes, please go ahead.

Previous Next

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