[libcxx-commits] [PATCH] D152381: [libc++] Extract ErrorHandler and related utilities from filesystem_common

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jun 13 11:43:59 PDT 2023


ldionne added inline comments.


================
Comment at: libcxx/src/filesystem/error.h:17
+#include <cstdint>
+#include <filesystem>
+#include <string>
----------------
Mordante wrote:
> Would it be possible to move the Windows specific code to a separate header, for example `windows_error.h`. I've used `make_windows_error` in `<print>`, where the CI fails if filesystem is disabled.
I think this header should work even when the platform doesn't have support for a filesystem, no?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152381



More information about the libcxx-commits mailing list