Page MenuHomeFreeBSD
Differential D44503

libc: Improve description of mktime() / timegm().
ClosedPublic
Actions

Authored by des on Mar 26 2024, 3:37 AM.
Tags
None
Referenced Files
F132262246: D44503.diff
Wed, Oct 15, 7:21 AM
Unknown Object (File)
Sun, Oct 12, 5:09 PM
Unknown Object (File)
Sat, Oct 11, 10:07 AM
Unknown Object (File)
Sat, Oct 11, 2:23 AM
Unknown Object (File)
Sat, Oct 11, 2:23 AM
Unknown Object (File)
Sat, Oct 11, 2:23 AM
Unknown Object (File)
Fri, Oct 10, 7:39 PM
Unknown Object (File)
Sep 15 2025, 12:20 PM
View All 66 Files
Subscribers
gbe
imp
pauamma_gundo.com

Details

Reviewers
pauamma_gundo.com
gbe
Group Reviewers
manpages
Commits
rGfdde01c5b449: libc: Improve description of mktime() / timegm().
rG7534109d13a6: libc: Improve description of mktime() / timegm().
Summary
  • Mention that mktime() and timegm() set errno on failure.
  • Correctly determining whether mktime() / timegm() succeeded with arbitrary input (where -1 can be a valid result) is non-trivial. Document the recommended procedure.

PR: 277863
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Mar 26 2024, 3:37 AM
Herald added a subscriber: imp. · View Herald TranscriptMar 26 2024, 3:37 AM
des requested review of this revision.Mar 26 2024, 3:37 AM
Harbormaster completed remote builds in B56779: Diff 136210.Mar 26 2024, 3:37 AM
pauamma_gundo.com accepted this revision.Mar 26 2024, 3:44 AM
pauamma_gundo.com added a subscriber: pauamma_gundo.com.

Grammar and spelling LGTM. Correctness (or portability) of the recommended error-checking is beyond my ken.

This revision is now accepted and ready to land.Mar 26 2024, 3:44 AM
gbe accepted this revision.Mar 26 2024, 6:08 AM
gbe added a subscriber: gbe.

LGTM

des added a comment.Mar 26 2024, 11:26 AM
In D44503#1015085, @pauamma_gundo.com wrote:

Correctness (or portability) of the recommended error-checking is beyond my ken.

Don't worry, I take full responsibility for that :)

https://mm.icann.org/pipermail/tz/2024-March/058822.html

Closed by commit rG7534109d13a6: libc: Improve description of mktime() / timegm(). (authored by des). · Explain WhyMar 27 2024, 11:27 AM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG7534109d13a6: libc: Improve description of mktime() / timegm()..
des mentioned this in rG662dc74155ad: tzcode: Clean up the ctime(3) manual page..Apr 4 2024, 11:40 AM
des added a commit: rGfdde01c5b449: libc: Improve description of mktime() / timegm()..

Revision Contents
Changeset List

PathSize
lib/
libc/
stdtime/
18 lines

Diff 136257

View Options

lib/libc/stdtime/ctime.3

Loading...