[PATCH] D111875: [Support] [Windows] Manually clean up temp files on network shares

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/lib/Support/Path.cpp:1212
+  std::error_code RemoveEC;
+  if (DeleteOnClose && !TmpName.empty()) {
+    RemoveEC = fs::remove(TmpName);
----------------
I'm not really liking the duplicate code between this and the below non-Windows block. Is there a way we could avoid this duplication?


================
Comment at: llvm/lib/Support/Path.cpp:1249-1252
+      Delete = true;
+      setDeleteDisposition(H, Delete);
+      if (!Delete)
+        DeleteOnClose = true;
----------------
Similar comment to above: this code is duplicated a few lines down. Perhaps another function, or maybe even mode the code into setDeleteDisposition somehow?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111875