Opened 15 years ago

Last modified 13 years ago

#90 new defect

innreport mishandles leap years and daylight savings time

Reported by: eagle Owned by: eagle
Priority: medium Milestone:
Component: utilities Version:
Severity: normal Keywords:
Cc:

Description

Alexander Bartolich reports that innreport mishandles leap years and daylight savings time. In an ​inn-workers posting he includes a modified version of its time parsing code that performs faster and more correctly (although still may not handle daylight savings time) and benchmarking of it, the current version, and the native Perl function (which is extremely slow due to constantly rechecking /etc/localtime).

Change History (3)

comment:1 Changed 15 years ago by Julien ÉLIE

Component: general → utilities

By the way, is

substr("000031059090120151181212243273304334", $dm * 3, 3)

right for bissextile years in innreport?

comment:2 Changed 15 years ago by Julien ÉLIE

(In [8280]) * Use of a modified version of time parsing code that performs

faster and more correctly.

  • Fix the parsing during leap years.

Thanks to Alexander Bartolich for the patch.

see #90
(Daylight savings time are still mishandled.)

comment:3 Changed 13 years ago by Julien ÉLIE

Summary: innreport mishandles leap years → innreport mishandles leap years and daylight savings time
Note: See TracTickets for help on using tickets.