GNU bug report logs - #71572
[PATCH] seconds-to-string-approximate

Previous Next

Full log

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

From: john muhl <jm <at> pub.pink>
To: JD Smith <jdtsmith <at> gmail.com>
Cc: Adam Porter <adam <at> alphapapa.net>, 71572 <at> debbugs.gnu.org,
 Eli Zaretskii <eliz <at> gnu.org>, Jonas Bernoulli <jonas <at> bernoul.li>,
 Paul Eggert <eggert <at> cs.ucla.edu>
Subject: Re: bug#71572: [PATCH] seconds-to-string-approximate
Date: Sat, 07 Dec 2024 13:17:57 -0600

JD Smith <jdtsmith <at> gmail.com> writes:

>> On Dec 7, 2024, at 8:02 AM, Eli Zaretskii <eliz <at> gnu.org> wrote:
>> 
>>> From: JD Smith <jdtsmith <at> gmail.com>
>>> Date: Sat, 30 Nov 2024 13:58:52 -0500
>>> Cc: Eli Zaretskii <eliz <at> gnu.org>,
>>> Adam Porter <adam <at> alphapapa.net>,
>>> jonas <at> bernoul.li,
>>> Paul Eggert <eggert <at> cs.ucla.edu>
>>> 
>>> I was recently reminded of the need for a more capable seconds-to-string.  
>>> 
>>> Anyone have any additional comments on this proposed patch? If not,
>>> I'd suggest someone with access
>>> merges.
>> 
>> A few minor nits below, and then we can install:
>
> Thanks.  Updated patch below.

Thanks for working on this. I gave a quick try and noticed some
amounts aren’t pluralized how I expect; e.g.

  (seconds-to-string 36541462 'expanded nil 1)
  ;; "1 year 1.9 month"

  (seconds-to-string 73082924 'expanded nil 2)
  ;; "2 year 3.79 months"

  (seconds-to-string 2511822 'expanded nil 3)
  ;; "4 week 1.072 day"

I would expect those to output:

  1 year 1.9 months
  2 years 3.79 months
  4 weeks 1.072 days

According to the Chicago Manual of Style all fractional values are
plural, even 1.0, 2.0 &c.

The incorrect “2 year”, “4 week” only happen when PRECISION is
non-nil; e.g.

  (seconds-to-string 2511822 'expanded nil)
  "4 weeks 1 day"

p.s. There is a missing “is” in the last sentence of the docstring
for seconds-to-string.

Previous Next

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