Skip to content

Conversation

@eldakesh-ms
Copy link
Contributor

Adds %j support to all the classes that can benefit from it.
This means all year_month_day* but and month_last (but only for
January).

Changes %aAuw to throw on invalid weekdays, but still work if the
weekday is explicitly ok (so a bad year_month_day will always throw but
a bad year_month_weekday can still print the weekday if that portion is
ok).

Adds %j support to all the classes that can benefit from it.
This means all year_month_day* but and month_last (but only for
January).

Changes %aAuw to throw on invalid weekdays, but still work if the
weekday is explicitly ok (so a bad year_month_day will always throw but
a bad year_month_weekday can still print the weekday if that portion is
ok).
@eldakesh-ms eldakesh-ms requested a review from a team as a code owner April 20, 2021 15:01
@mnatsuhara mnatsuhara added the chrono C++20 chrono label Apr 20, 2021
Copy link
Contributor

@mnatsuhara mnatsuhara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One lingering question, but otherwise looks good!

@eldakesh-ms eldakesh-ms merged commit 65ad8a7 into microsoft:chronat2 Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chrono C++20 chrono

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants