[Lldb-commits] [lldb] r340652 - Fixed windows bots that were failing because of PATH_MAX
Raphael Isemann via lldb-commits
lldb-commits at lists.llvm.org
Fri Aug 24 13:55:23 PDT 2018
Author: teemperor
Date: Fri Aug 24 13:55:23 2018
New Revision: 340652
URL: http://llvm.org/viewvc/llvm-project?rev=340652&view=rev
Log:
Fixed windows bots that were failing because of PATH_MAX
As we only use PATH_MAX for an assert in a unit test that is supposed
to catch the random failures on the Swift CI bots, we might as well
just ifdef this assert out on Windows.
Modified:
lldb/trunk/unittests/Interpreter/TestCompletion.cpp
Modified: lldb/trunk/unittests/Interpreter/TestCompletion.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/unittests/Interpreter/TestCompletion.cpp?rev=340652&r1=340651&r2=340652&view=diff
==============================================================================
--- lldb/trunk/unittests/Interpreter/TestCompletion.cpp (original)
+++ lldb/trunk/unittests/Interpreter/TestCompletion.cpp Fri Aug 24 13:55:23 2018
@@ -165,7 +165,9 @@ TEST_F(CompletionTest, DirCompletionAbso
// for that directory, not items under it.
// Sanity check that the path we complete on exists and isn't too long.
ASSERT_TRUE(llvm::sys::fs::exists(BaseDir));
+#ifdef PATH_MAX
ASSERT_LE(BaseDir.size(), static_cast<size_t>(PATH_MAX));
+#endif
size_t Count =
CommandCompletions::DiskDirectories(BaseDir, Results, Resolver);
ASSERT_EQ(1u, Count);
More information about the lldb-commits
mailing list