[libcxx-commits] [libcxx] [libc++] Implement LWG3430 disallow implicit conversion of the source arguments to `std::filesystem::path` when constructing `std::basic_*fstream` (PR #85079)

[Mark de Wever via libcxx-commits]

================
@@ -158,8 +158,8 @@ public:
     basic_fstream();
     explicit basic_fstream(const char* s, ios_base::openmode mode = ios_base::in|ios_base::out);
     explicit basic_fstream(const string& s, ios_base::openmode mode = ios_base::in|ios_base::out);
-    explicit basic_fstream(const filesystem::path& p,
-                           ios_base::openmode mode = ios_base::in|ios_base::out); C++17
+    template<class T>
+    explicit basic_fstream(const T& s, ios_base::openmode mode = ios_base::in | ios_base::out); // Since C++17
----------------
mordante wrote:

It's indeed correct this may break existing code. Louis tested this at Apple, based on the [results](
https://github.com/llvm/llvm-project/pull/85079#issuecomment-2032096059) we applied the possibly breaking change. Is the breakage for you worse?

https://github.com/llvm/llvm-project/pull/85079