cplusplus / cplusplus/draft

[time.format] incorrect namespace qualifications

Open
#5,929 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

P2-Bug
Dominant language
TeX
Stars
221
Forks
813
Avg merge
16h 4m
Merged PRs (30d)
36

Description

p1 and p4 refer to std::chrono::duration which should just be chrono::duration.

p10 and p11 refer to sys_time which should be chrono::sys_time.

p12 is missing the chrono:: throughout:

equivalent to those formatted by a sys_­time initialized with clock_­cast<system_­clock>(t), or by a utc_­time initialized with clock_­cast<utc_­clock>(t)

p13 is says local_time which should be chrono::local_time

The declarations before p14 are a struct and a function in chrono whereas the surrounding definitions are all in namespace std. They should be shown as chrono::local-time-format-t and chrono::local_time_format(...).

p15 should refer to chrono::local-time-format-t

p17 should refer to chrono::sys_time.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue names no file or test; locate the time.format wording in the C++ draft source and inspect paragraphs p1–p17. Compare each listed name with the surrounding namespace declarations, then verify that all requested qualifications appear correctly in the rendered draft.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, tex
Domain
documentation
Issue type
Documentation
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.