[PATCH] D27579: [libFuzzer] Diff 19 - Automatically link with Windows library.

Adrian McCarthy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 11:15:57 PST 2016


amccarth added a comment.

LGTM.

Please update the title and description of this patch to reflect the new direction.



================
Comment at: lib/Fuzzer/FuzzerIOWindows.cpp:261
+  if (DirLen) {
+    --DirLen; // Remove trailing '\'.
+    if (!FileLen) { // Path ended in '\'.
----------------
Nit:  In these comments, I would say `separator` instead of `'\'`.  It removes both slashes and backslashes, and `'\'` looks like a bad char literal.


Repository:
  rL LLVM

https://reviews.llvm.org/D27579





More information about the llvm-commits mailing list