[Lldb-commits] [lldb] 7a78152 - [lldb] Fix a warning

Kazu Hirata via lldb-commits lldb-commits at lists.llvm.org
Tue Feb 18 08:10:43 PST 2025


Author: Kazu Hirata
Date: 2025-02-18T08:10:38-08:00
New Revision: 7a781527682ebe90616b05fb2311b1d0d6fc997e

URL: https://github.com/llvm/llvm-project/commit/7a781527682ebe90616b05fb2311b1d0d6fc997e
DIFF: https://github.com/llvm/llvm-project/commit/7a781527682ebe90616b05fb2311b1d0d6fc997e.diff

LOG: [lldb] Fix a warning

This patch fixes:

  third-party/unittest/googletest/include/gtest/gtest.h:1379:11:
  error: comparison of integers of different signs: 'const int' and
  'const unsigned long' [-Werror,-Wsign-compare]

Added: 
    

Modified: 
    lldb/unittests/Core/TelemetryTest.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/unittests/Core/TelemetryTest.cpp b/lldb/unittests/Core/TelemetryTest.cpp
index 03bd8a59ba7dc..0f2eaccb21a2c 100644
--- a/lldb/unittests/Core/TelemetryTest.cpp
+++ b/lldb/unittests/Core/TelemetryTest.cpp
@@ -87,7 +87,7 @@ TEST(TelemetryTest, PluginTest) {
   entry.msg = "";
 
   ASSERT_THAT_ERROR(ins->dispatch(&entry), ::llvm::Succeeded());
-  ASSERT_EQ(1, expected_entries.size());
+  ASSERT_EQ(1U, expected_entries.size());
   EXPECT_EQ("In FakePlugin",
             llvm::dyn_cast<lldb_private::FakeTelemetryInfo>(expected_entries[0])
                 ->msg);


        


More information about the lldb-commits mailing list