Skip to content

Commit 4b4f224

Browse files
committed
Implement %j specifier for std::chrono::duration
This adds support for `%j` presentation type for duration types: > "If the type being formatted is a specialization of duration, the decimal number of days without padding." Fixes fmtlib#3643.
1 parent 69f01fe commit 4b4f224

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

include/fmt/chrono.h

+7-1
Original file line numberDiff line numberDiff line change
@@ -1622,6 +1622,7 @@ struct chrono_format_checker : null_chrono_spec_handler<chrono_format_checker> {
16221622

16231623
template <typename Char>
16241624
FMT_CONSTEXPR void on_text(const Char*, const Char*) {}
1625+
FMT_CONSTEXPR void on_day_of_year() {}
16251626
FMT_CONSTEXPR void on_24_hour(numeric_system, pad_type) {}
16261627
FMT_CONSTEXPR void on_12_hour(numeric_system, pad_type) {}
16271628
FMT_CONSTEXPR void on_minute(numeric_system, pad_type) {}
@@ -1806,6 +1807,7 @@ struct chrono_formatter {
18061807
return true;
18071808
}
18081809

1810+
Rep days() const { return static_cast<Rep>(s.count() / 86400); }
18091811
Rep hour() const { return static_cast<Rep>(mod((s.count() / 3600), 24)); }
18101812

18111813
Rep hour12() const {
@@ -1884,10 +1886,14 @@ struct chrono_formatter {
18841886
void on_dec0_week_of_year(numeric_system) {}
18851887
void on_dec1_week_of_year(numeric_system) {}
18861888
void on_iso_week_of_year(numeric_system) {}
1887-
void on_day_of_year() {}
18881889
void on_day_of_month(numeric_system) {}
18891890
void on_day_of_month_space(numeric_system) {}
18901891

1892+
void on_day_of_year() {
1893+
if (handle_nan_inf()) return;
1894+
return write(days(), 0);
1895+
}
1896+
18911897
void on_24_hour(numeric_system ns, pad_type pad) {
18921898
if (handle_nan_inf()) return;
18931899

test/chrono-test.cc

+2
Original file line numberDiff line numberDiff line change
@@ -537,6 +537,8 @@ TEST(chrono_test, format_specs) {
537537
EXPECT_EQ("12", fmt::format("{:%I}", std::chrono::hours(24)));
538538
EXPECT_EQ("04", fmt::format("{:%I}", std::chrono::hours(4)));
539539
EXPECT_EQ("02", fmt::format("{:%I}", std::chrono::hours(14)));
540+
EXPECT_EQ("12345", fmt::format("{:%j}", days(12345)));
541+
EXPECT_EQ("12345", fmt::format("{:%j}", std::chrono::hours(12345*24+12)));
540542
EXPECT_EQ("03:25:45",
541543
fmt::format("{:%H:%M:%S}", std::chrono::seconds(12345)));
542544
EXPECT_EQ("03:25", fmt::format("{:%R}", std::chrono::seconds(12345)));

0 commit comments

Comments
 (0)