June 12, 2017
10:57 p.m.
On 06/12/2017 03:22 PM, Robert Elz wrote:
zdump might, but the code in question (in asctime) obviously doesn't
True. To some extent I removed that comment for reasons that I didn't bother to write down in the commit log. First, snprintf has been required since C89 and is now universal, and there's little point to documenting ancient backward-compatibility problems that no longer matter. Second, sprintf can be preferable to snprintf when either will do, as sprintf has a simpler API - i.e., nowadays the code avoids snprintf for reasons other than the just-removed comment.