Simplify range formatter

This commit is contained in:
Victor Zverovich
2024-03-09 08:20:27 -08:00
parent 11f2f30f0b
commit e281749c5d
2 changed files with 12 additions and 10 deletions

View File

@ -2961,8 +2961,8 @@ template <typename OutputIt, typename Sentinel = OutputIt>
struct format_to_result { struct format_to_result {
/** Iterator pointing to just after the last successful write in the range. */ /** Iterator pointing to just after the last successful write in the range. */
OutputIt out; OutputIt out;
/** Sentinel indicating the end of the output range. */ /** Specifies if the output was truncated. */
Sentinel out_last; bool truncated;
FMT_CONSTEXPR operator OutputIt&() & noexcept { return out; } FMT_CONSTEXPR operator OutputIt&() & noexcept { return out; }
FMT_CONSTEXPR operator const OutputIt&() const& noexcept { return out; } FMT_CONSTEXPR operator const OutputIt&() const& noexcept { return out; }
@ -2974,13 +2974,15 @@ struct format_to_result {
template <size_t N> template <size_t N>
auto vformat_to(char (&out)[N], string_view fmt, format_args args) auto vformat_to(char (&out)[N], string_view fmt, format_args args)
-> format_to_result<char*> { -> format_to_result<char*> {
return {vformat_to_n(out, N, fmt, args).out, out + N}; auto result = vformat_to_n(out, N, fmt, args);
return {result.out, result.size > N};
} }
template <size_t N, typename... T> template <size_t N, typename... T>
FMT_INLINE auto format_to(char (&out)[N], format_string<T...> fmt, T&&... args) FMT_INLINE auto format_to(char (&out)[N], format_string<T...> fmt, T&&... args)
-> format_to_result<char*> { -> format_to_result<char*> {
return vformat_to(out, fmt, fmt::make_format_args(args...)); auto result = fmt::format_to_n(out, N, fmt, static_cast<T&&>(args)...);
return {result.out, result.size > N};
} }
/** Returns the number of chars in the output of ``format(fmt, args...)``. */ /** Returns the number of chars in the output of ``format(fmt, args...)``. */

View File

@ -697,13 +697,13 @@ TEST(core_test, format_to_c_array) {
char buffer[4]; char buffer[4];
auto result = fmt::format_to(buffer, "{}", 12345); auto result = fmt::format_to(buffer, "{}", 12345);
EXPECT_EQ(4, std::distance(&buffer[0], result.out)); EXPECT_EQ(4, std::distance(&buffer[0], result.out));
EXPECT_EQ(0, std::distance(result.out, result.out_last)); EXPECT_TRUE(result.truncated);
EXPECT_EQ(buffer + 4, result.out); EXPECT_EQ(buffer + 4, result.out);
EXPECT_EQ("1234", fmt::string_view(buffer, 4)); EXPECT_EQ("1234", fmt::string_view(buffer, 4));
result = fmt::format_to(buffer, "{:s}", "foobar"); result = fmt::format_to(buffer, "{:s}", "foobar");
EXPECT_EQ(4, std::distance(&buffer[0], result.out)); EXPECT_EQ(4, std::distance(&buffer[0], result.out));
EXPECT_EQ(0, std::distance(result.out, result.out_last)); EXPECT_TRUE(result.truncated);
EXPECT_EQ(buffer + 4, result.out); EXPECT_EQ(buffer + 4, result.out);
EXPECT_EQ("foob", fmt::string_view(buffer, 4)); EXPECT_EQ("foob", fmt::string_view(buffer, 4));
@ -713,24 +713,24 @@ TEST(core_test, format_to_c_array) {
buffer[3] = 'x'; buffer[3] = 'x';
result = fmt::format_to(buffer, "{}", 'A'); result = fmt::format_to(buffer, "{}", 'A');
EXPECT_EQ(1, std::distance(&buffer[0], result.out)); EXPECT_EQ(1, std::distance(&buffer[0], result.out));
EXPECT_EQ(3, std::distance(result.out, result.out_last)); EXPECT_FALSE(result.truncated);
EXPECT_EQ(buffer + 1, result.out); EXPECT_EQ(buffer + 1, result.out);
EXPECT_EQ("Axxx", fmt::string_view(buffer, 4)); EXPECT_EQ("Axxx", fmt::string_view(buffer, 4));
result = fmt::format_to(buffer, "{}{} ", 'B', 'C'); result = fmt::format_to(buffer, "{}{} ", 'B', 'C');
EXPECT_EQ(3, std::distance(&buffer[0], result.out)); EXPECT_EQ(3, std::distance(&buffer[0], result.out));
EXPECT_EQ(1, std::distance(result.out, result.out_last)); EXPECT_FALSE(result.truncated);
EXPECT_EQ(buffer + 3, result.out); EXPECT_EQ(buffer + 3, result.out);
EXPECT_EQ("BC x", fmt::string_view(buffer, 4)); EXPECT_EQ("BC x", fmt::string_view(buffer, 4));
result = fmt::format_to(buffer, "{}", "ABCDE"); result = fmt::format_to(buffer, "{}", "ABCDE");
EXPECT_EQ(4, std::distance(&buffer[0], result.out)); EXPECT_EQ(4, std::distance(&buffer[0], result.out));
EXPECT_EQ(0, std::distance(result.out, result.out_last)); EXPECT_TRUE(result.truncated);
EXPECT_EQ("ABCD", fmt::string_view(buffer, 4)); EXPECT_EQ("ABCD", fmt::string_view(buffer, 4));
result = fmt::format_to(buffer, "{}", std::string(1000, '*')); result = fmt::format_to(buffer, "{}", std::string(1000, '*'));
EXPECT_EQ(4, std::distance(&buffer[0], result.out)); EXPECT_EQ(4, std::distance(&buffer[0], result.out));
EXPECT_EQ(0, std::distance(result.out, result.out_last)); EXPECT_TRUE(result.truncated);
EXPECT_EQ("****", fmt::string_view(buffer, 4)); EXPECT_EQ("****", fmt::string_view(buffer, 4));
} }