GNU bug report logs - #56486
[PATCH] * time-date.el (decoded-time-period): Rename TIME in docstring

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 56486 in the body.
You can then email your comments to 56486 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: Philip Kaludercic <philipk <at> posteo.net>
To: bug-gnu-emacs <at> gnu.org
Subject: [PATCH] * time-date.el (decoded-time-period): Rename TIME in docstring
Date: Sun, 10 Jul 2022 18:55:32 +0000

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

Tags: patch


This appears to be a typo?


In GNU Emacs 29.0.50 (build 5, x86_64-pc-linux-gnu, GTK+ Version 3.24.34, cairo version 1.17.6)
 of 2022-07-04 built on rhea
Repository revision: 09444b5fc4d9703eddd227df192603fd03b45577
Repository branch: master
System Description: Fedora Linux 36 (Workstation Edition)

Configured using:
 'configure --with-pgtk --with-imagemagick --with-native-compilation
 'CFLAGS=-march=native -O2 -pipe' LDFLAGS=-flto'

[0001-time-date.el-decoded-time-period-Rename-TIME-in-docs.patch (text/patch, attachment)]

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

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Philip Kaludercic <philipk <at> posteo.net>, 56486 <at> debbugs.gnu.org
Subject: Re: bug#56486: [PATCH] * time-date.el (decoded-time-period): Rename
 TIME in docstring
Date: Sun, 10 Jul 2022 14:36:47 -0700

Philip Kaludercic <philipk <at> posteo.net> writes:

> This appears to be a typo?

Indeed, please push to master.

> diff --git a/lisp/calendar/time-date.el b/lisp/calendar/time-date.el
> index d19134db83..d1afd8ce95 100644
> --- a/lisp/calendar/time-date.el
> +++ b/lisp/calendar/time-date.el
> @@ -590,7 +590,7 @@ decoded-time-set-defaults
>    time)
>
>  (defun decoded-time-period (time)
> -  "Interpret DECODED as a period and return its length in seconds.
> +  "Interpret TIME as a period and return its length in seconds.
>  For computational purposes, years are 365 days long and months
>  are 30 days long."
>    (+ (if (consp (decoded-time-second time))
> --
> 2.36.1

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

From: Philip Kaludercic <philipk <at> posteo.net>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: 56486-done <at> debbugs.gnu.org
Subject: Re: bug#56486: [PATCH] * time-date.el (decoded-time-period): Rename
 TIME in docstring
Date: Sun, 10 Jul 2022 23:07:01 +0000

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

> Philip Kaludercic <philipk <at> posteo.net> writes:
>
>> This appears to be a typo?
>
> Indeed, please push to master.

Done.

Previous Next

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