[libcxx-commits] [PATCH] D146398: [libcxx] Fix using std::wcout/wcin on Windows with streams configured in wide mode

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Apr 18 11:33:17 PDT 2023


Mordante added inline comments.


================
Comment at: libcxx/src/std_stream.h:66
+#else
+    inline static constexpr bool __is_win32api_wide_char = false;
+#endif
----------------
mstorsjo wrote:
> Mordante wrote:
> > I actually intended it not to be in the class. I don't mind it, but then we don't need the inline.
> Yeah, it looked like that, but I wasn’t sure of how that’d work with the template parameter `CharT` - wouldn’t I need to pass it on from each of the class templates to the template bool variable then? (This is a bit outside of the C++ template syntax I’m proficient about…)
Yes then you needed to pass them in `if constexpr`. But this in fine too (modulo the inline).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146398/new/

https://reviews.llvm.org/D146398



More information about the libcxx-commits mailing list