On 7/18/22 09:03, Paul Eggert wrote:
I suppose someone should file a defect report with the C standardization committee.
I found what I hope is the correct email address for that (it's not well-advertised) and submitted the following bug report: ---- Subject: strftime %z and %Z depend on more than just tm_isdst A recent discussion in the Time Zone Database mailing list <https://mm.icann.org/pipermail/tz/2022-July/031674.html> has prompted me to file this bug report against the C standard. Draft N2912, section 7.28.3.5 "The strftime function", pages 365-6, paragraph 3 says that the %z and %Z conversion specifiers examine only the tm_isdst members of the passed-in structure. However, this is not how many implementations actually behave. Some implementations (e.g., AIX, Solaris) infer %z and %Z output from tm_year, tm_mon, tm_mday, tm_hour, tm_min, tm_sec, and tm_isdst, and therefore sometimes guess %z or %Z output incorrectly when the clock moves back from one standard time to another (e.g., Iran at the end of the year 1978). These implementations typically also depend on the current setting of the TZ environment variable. Other implementations (e.g., FreeBSD, GNU/Linux) get the UT offset and abbreviation from struct tm members tm_gmtoff and tm_zone that are not specified by the C standard. These implementations do not have to guess %z and %Z output; however, they require that the nonstandard members be filled in correctly by localtime or equivalent. Implementations that conform to the standard's current wording cannot handle anything more complicated than a time zone with just one standard time offset and abbreviation, which means they cannot handle timestamps for locations like Iran, Portugal, etc., that have changed their standard-time offset. The C standard should not require implementations to be so limited that they cannot handle common timekeeping situations. A simple fix for this issue is to require applications to initialize a struct tm as if by localtime or equivalent, before passing the struct tm to strftime. This is what applications need to do anyway, if they want to be portable to real-world systems such as AIX, FreeBSD, GNU/Linux, Solaris, etc. Proposed fix: 1. Remove the two instances of "[tm_isdst]" in the %z and %Z entries in Draft N2912, section 7.28.3.5 "The strftime function", pages 365-6, paragraph 3. 2. Add the following text to that paragraph: If the %z and %Z conversion specifiers are used, the broken-down time structure pointed to by timeptr shall contain values generated by a successful previous call to gmtime, gmtime_r, gmtime_s, localtime, localtime_r, localtime_s, or mktime.