GNU bug report logs -
#74841
[PATCH] srfi-19: Fix ~V converter in date->string.
Previous Next
Reported by: Tomas Volf <~@wolfsden.cz>
Date: Thu, 12 Dec 2024 21:05:02 UTC
Severity: normal
Tags: patch
Done: Ludovic Courtès <ludo <at> gnu.org>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
[Message part 1 (text/plain, inline)]
Your bug report
#74841: [PATCH] srfi-19: Fix ~V converter in date->string.
which was filed against the guile package, has been closed.
The explanation is attached below, along with your original report.
If you require more details, please reply to 74841 <at> debbugs.gnu.org.
--
74841: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=74841
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
Tomas Volf <~@wolfsden.cz> skribis:
> The ~V is supposed to print ISO week number, not a week number. This
> commit fixes that.
>
> * module/srfi/srfi-19.scm (date-week-number-iso): New procedure taken
> from the reference implementation.
> (directives)<#\V>: Use it.
> * test-suite/tests/srfi-19.test ("date->string ~V"): Add tests taken
> from the reference test suite.
> * doc/ref/srfi-modules.texi (SRFI-19 Date to string): Mention ISO-8601
> in description for ~V.
Applied, thanks!
[Message part 3 (message/rfc822, inline)]
The ~V is supposed to print ISO week number, not a week number. This
commit fixes that.
* module/srfi/srfi-19.scm (date-week-number-iso): New procedure taken
from the reference implementation.
(directives)<#\V>: Use it.
* test-suite/tests/srfi-19.test ("date->string ~V"): Add tests taken
from the reference test suite.
---
module/srfi/srfi-19.scm | 21 ++++++++++++-
test-suite/tests/srfi-19.test | 57 ++++++++++++++++++++++++++++++++++-
2 files changed, 76 insertions(+), 2 deletions(-)
diff --git a/module/srfi/srfi-19.scm b/module/srfi/srfi-19.scm
index d809ac1ec..77be57a0e 100644
--- a/module/srfi/srfi-19.scm
+++ b/module/srfi/srfi-19.scm
@@ -753,6 +753,25 @@
(days-before-first-week date day-of-week-starting-week))
7))
+(define (date-week-number-iso date)
+ ;; The week with the year's first Thursday is week 01.
+ (let* ((first-day-of-the-week (week-day 1 1 (date-year date)))
+ (offset (if (> first-day-of-the-week 4) 0 1))
+ ;; -2: decrement one day to compensate 1-origin of date-year-day,
+ ;; and decrement one more day for Sunday belongs to the previous week.
+ (w (+ (floor-quotient (+ (date-year-day date) first-day-of-the-week -2)
+ 7)
+ offset)))
+ (cond ((zero? w)
+ ;; date belongs to the last week of the previous year
+ (date-week-number-iso (make-date 0 0 0 0 31 12
+ (- (date-year date) 1) 0)))
+ ((and (= w 53)
+ (<= (week-day 1 1 (+ (date-year date) 1)) 4))
+ ;; date belongs to the first week of the next year
+ 1)
+ (else w))))
+
(define (current-date . tz-offset)
(let ((time (current-time time-utc)))
(time-utc->date
@@ -1043,7 +1062,7 @@
(display (padding (date-week-number date 0)
#\0 2) port))))
(cons #\V (lambda (date pad-with port)
- (display (padding (date-week-number date 1)
+ (display (padding (date-week-number-iso date)
#\0 2) port)))
(cons #\w (lambda (date pad-with port)
(display (date-week-day date) port)))
diff --git a/test-suite/tests/srfi-19.test b/test-suite/tests/srfi-19.test
index 55eb82320..5e3e1f445 100644
--- a/test-suite/tests/srfi-19.test
+++ b/test-suite/tests/srfi-19.test
@@ -412,7 +412,62 @@ incomplete numerical tower implementation.)"
(with-test-prefix "date-week-number"
(pass-if (= 0 (date-week-number (make-date 0 0 0 0 1 1 1984 0) 0)))
(pass-if (= 0 (date-week-number (make-date 0 0 0 0 7 1 1984 0) 0)))
- (pass-if (= 1 (date-week-number (make-date 0 0 0 0 8 1 1984 0) 0)))))
+ (pass-if (= 1 (date-week-number (make-date 0 0 0 0 8 1 1984 0) 0))))
+
+ (let ((convert (λ (lst)
+ (date->string
+ (make-date 0 0 0 0
+ (caddr lst) (cadr lst) (car lst)
+ 0)
+ "~V"))))
+ (with-test-prefix "date->string ~V"
+ (pass-if-equal "Thursday, week 53" "53"
+ (convert '(2020 12 31)))
+ (pass-if-equal "Friday, week 53 (previous year)" "53"
+ (convert '(2021 1 1)))
+ (pass-if-equal "Sunday, week 53 (previous year)" "53"
+ (convert '(2021 1 3)))
+ (pass-if-equal "Monday, week 1" "01"
+ (convert '(2021 1 4)))
+
+ (pass-if-equal "Sunday, week 52" "52"
+ (convert '(2019 12 29)))
+ (pass-if-equal "Monday, week 1 (next year)" "01"
+ (convert '(2019 12 30)))
+ (pass-if-equal "Tuesday, week 1 (next year)" "01"
+ (convert '(2019 12 31)))
+ (pass-if-equal "Wednesday, week 1" "01"
+ (convert '(2020 1 1)))
+
+ (pass-if-equal "Saturday, week 52" "52"
+ (convert '(2016 12 31)))
+ (pass-if-equal "Sunday, week 52 (previous year)" "52"
+ (convert '(2017 1 1)))
+ (pass-if-equal "Monday, week 1" "01"
+ (convert '(2017 1 2)))
+ (pass-if-equal "Sunday, week 1" "01"
+ (convert '(2017 1 8)))
+ (pass-if-equal "Monday, week 2" "02"
+ (convert '(2017 1 9)))
+
+ (pass-if-equal "Sunday, week 52" "52"
+ (convert '(2014 12 28)))
+ (pass-if-equal "Monday, week 1 (next year)" "01"
+ (convert '(2014 12 29)))
+ (pass-if-equal "Tuesday, week 1 (next year)" "01"
+ (convert '(2014 12 30)))
+ (pass-if-equal "Wednesday, week 1 (next year)" "01"
+ (convert '(2014 12 31)))
+ (pass-if-equal "Thursday, week 1" "01"
+ (convert '(2015 1 1)))
+ (pass-if-equal "Friday, week 1" "01"
+ (convert '(2015 1 2)))
+ (pass-if-equal "Saturday, week 1" "01"
+ (convert '(2015 1 3)))
+ (pass-if-equal "Sunday, week 1" "01"
+ (convert '(2015 1 4)))
+ (pass-if-equal "Monday, week 2" "02"
+ (convert '(2015 1 5))))))
;; Local Variables:
--
2.46.0
This bug report was last modified 135 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.