commit 00588945db9fa0a322efea244ed0f503033b5ba7
author Ed Tanous <ed@tanous.net> Mon May 19 09:28:05 2025 -0700
committer Andrew Jeffery <andrew@codeconstruct.com.au> Thu May 29 01:27:41 2025 +0000
tree 353828c75eba27f3488ad64169e329f2c7d524c8
parent 78ecfc4cf1da321d7d53cd5804b9fdd98c2593ed

Fix signed/unsigned comparisons

When built on x86-64, these comparisons trigger a warning.  Fix it by
making these constants have the correct signedness.

Change-Id: I56e0b6e4e14bffbc7e45b8264f78fb51f4116789
Signed-off-by: Ed Tanous <ed@tanous.net>

src/tests/test_Utils.cpp

diff --git a/src/tests/test_Utils.cpp b/src/tests/test_Utils.cpp
index 957a9e9..9f772e0 100644
--- a/src/tests/test_Utils.cpp
+++ b/src/tests/test_Utils.cpp
@@ -230,8 +230,8 @@
 
     auto ret = getDeviceBusAddr(devName, bus, provisionedId);
     EXPECT_TRUE(ret);
-    EXPECT_EQ(bus, 0);
-    EXPECT_EQ(provisionedId, 0x22400000001);
+    EXPECT_EQ(bus, 0U);
+    EXPECT_EQ(provisionedId, 0x22400000001U);
 }
 
 TEST(GetDeviceBusAddrTest, AllValid)
@@ -242,6 +242,6 @@
 
     auto ret = getDeviceBusAddr(devName, bus, addr);
     EXPECT_TRUE(ret);
-    EXPECT_EQ(bus, 12);
-    EXPECT_EQ(addr, 0xaf);
+    EXPECT_EQ(bus, 12U);
+    EXPECT_EQ(addr, 0xafU);
 }