Fix for issue fmtlib#3319.

Signed-off-by: Vladislav Shchapov <[email protected]>
phprus · Feb 25, 2023 · 3351f9d · 3351f9d
1 parent 5b83020
commit 3351f9d
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/include/fmt/std.h b/include/fmt/std.h
@@ -77,6 +77,10 @@ inline void write_escaped_path<std::filesystem::path::value_type>(
 template <typename Char>
 struct formatter<std::filesystem::path, Char>
     : formatter<basic_string_view<Char>> {
+  template <typename ParseContext> FMT_CONSTEXPR auto parse(ParseContext& ctx) {
+    this->set_debug_format(false);
+    return formatter<basic_string_view<Char>>::parse(ctx);
+  }
   template <typename FormatContext>
   auto format(const std::filesystem::path& p, FormatContext& ctx) const ->
       typename FormatContext::iterator {

diff --git a/test/std-test.cc b/test/std-test.cc
@@ -46,6 +46,20 @@ TEST(ranges_std_test, format_vector_path) {
 #endif
 }
 
+TEST(ranges_std_test, format_quote_path) {
+  // Test quoted problem described in #3319.
+#ifdef __cpp_lib_filesystem
+  auto v =
+      std::vector<std::filesystem::path>{"path1/file1.txt", "path2/file2.txt"};
+  EXPECT_EQ(fmt::format("{}", v), "[\"path1/file1.txt\", \"path2/file2.txt\"]");
+
+#  ifdef __cpp_lib_optional
+  auto o = std::optional<std::filesystem::path>("path/file.txt");
+  EXPECT_EQ(fmt::format("{}", o), "optional(\"path/file.txt\")");
+#  endif
+#endif
+}
+
 TEST(std_test, thread_id) {
   EXPECT_FALSE(fmt::format("{}", std::this_thread::get_id()).empty());
 }