Eliminate 'boolean result used in bitwise op' cppcheck false warning
authorIvan Maidanski <ivmai@mail.ru>
Tue, 19 Sep 2017 06:58:11 +0000 (09:58 +0300)
committerIvan Maidanski <ivmai@mail.ru>
Tue, 19 Sep 2017 07:00:57 +0000 (10:00 +0300)
* extra/AmigaOS.c [GC_AMIGA_DS] (GC_register_data_segments):
Replace (ULONG)&data[1] in comparison to (ULONG)(&data[1]).
* misc.c [!THREADS && !SPARC] (GC_do_blocking_inner):
Replace (ptr_t)&d in comparison to (ptr_t)(&d).

extra/AmigaOS.c
misc.c

index 549e20f..d8311ee 100644 (file)
@@ -106,8 +106,9 @@ ptr_t GC_get_main_stack_base(void)
 
         for (data = (ULONG *)BADDR(myseglist); data != NULL;
              data = (ULONG *)BADDR(data[0])) {
-          if (((ULONG) GC_register_data_segments < (ULONG) &data[1]) ||
-              ((ULONG) GC_register_data_segments > (ULONG) &data[1] + data[-1])) {
+          if ((ULONG)GC_register_data_segments < (ULONG)(&data[1])
+              || (ULONG)GC_register_data_segments > (ULONG)(&data[1])
+                                                    + data[-1]) {
 #             ifdef __GNUC__
                 if (dataSegSize == data[-1]) {
                   found_segment = TRUE;
diff --git a/misc.c b/misc.c
index 4ba0955..e4607f6 100644 (file)
--- a/misc.c
+++ b/misc.c
@@ -2031,7 +2031,7 @@ STATIC void GC_do_blocking_inner(ptr_t data, void * context GC_ATTR_UNUSED)
 #   ifdef SPARC
         GC_ASSERT(GC_blocked_sp != NULL);
 #   else
-        GC_ASSERT(GC_blocked_sp == (ptr_t) &d);
+        GC_ASSERT(GC_blocked_sp == (ptr_t)(&d));
 #   endif
 #   if defined(CPPCHECK)
       GC_noop1((word)GC_blocked_sp);