projects / reactos.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 8325a50)
[NETWORKING][CABMAN]
authorThomas Faber <thomas.faber@reactos.org>
Fri, 23 Oct 2015 07:26:42 +0000 (07:26 +0000)
committerThomas Faber <thomas.faber@reactos.org>
Fri, 23 Oct 2015 07:26:42 +0000 (07:26 +0000)
- Attempt to silence some 200 benign "Operands don't affect result" Coverity defects

svn path=/trunk/; revision=69652

reactos/dll/win32/ws2_32/include/debug.h patch | blob | history
reactos/dll/win32/wshirda/debug.h patch | blob | history
reactos/drivers/network/afd/include/debug.h patch | blob | history
reactos/drivers/network/dd/ne2000/include/debug.h patch | blob | history
reactos/drivers/network/dd/rtl8139/debug.h patch | blob | history
reactos/drivers/network/lan/include/debug.h patch | blob | history
reactos/drivers/network/ndis/include/debug.h patch | blob | history
reactos/tools/cabman/cabinet.h patch | blob | history
rosapps/applications/net/tditest/include/debug.h patch | blob | history

diff --git a/reactos/dll/win32/ws2_32/include/debug.h b/reactos/dll/win32/ws2_32/include/debug.h
index aa6d970..22cfd39 100644 (file)
--- a/reactos/dll/win32/ws2_32/include/debug.h
+++ b/reactos/dll/win32/ws2_32/include/debug.h
@@ -28,8 +28,9 @@
 extern DWORD DebugTraceLevel;
 
 #define WS_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%hS:%d)(%hS) ", __FILE__, __LINE__, __FUNCTION__); \
                DbgPrint _x_; \
     }
diff --git a/reactos/dll/win32/wshirda/debug.h b/reactos/dll/win32/wshirda/debug.h
index c8702f3..ebdca62 100644 (file)
--- a/reactos/dll/win32/wshirda/debug.h
+++ b/reactos/dll/win32/wshirda/debug.h
@@ -22,8 +22,9 @@
 extern DWORD DebugTraceLevel;
 
 #define WSH_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%hS:%d)(%hS) ", __FILE__, __LINE__, __FUNCTION__); \
                DbgPrint _x_; \
     }
diff --git a/reactos/drivers/network/afd/include/debug.h b/reactos/drivers/network/afd/include/debug.h
index f4ed29a..15377d0 100644 (file)
--- a/reactos/drivers/network/afd/include/debug.h
+++ b/reactos/drivers/network/afd/include/debug.h
@@ -26,8 +26,9 @@ extern DWORD DebugTraceLevel;
 #ifdef _MSC_VER
 
 #define AFD_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d) ", __FILE__, __LINE__); \
         DbgPrint _x_ ; \
     }
@@ -35,8 +36,9 @@ extern DWORD DebugTraceLevel;
 #else /* _MSC_VER */
 
 #define AFD_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d)(%s) ", __FILE__, __LINE__, __FUNCTION__); \
         DbgPrint _x_ ; \
     }
diff --git a/reactos/drivers/network/dd/ne2000/include/debug.h b/reactos/drivers/network/dd/ne2000/include/debug.h
index f495766..73e7189 100644 (file)
--- a/reactos/drivers/network/dd/ne2000/include/debug.h
+++ b/reactos/drivers/network/dd/ne2000/include/debug.h
@@ -25,8 +25,9 @@ extern ULONG DebugTraceLevel;
 #ifdef _MSC_VER
 
 #define NDIS_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d) ", __FILE__, __LINE__); \
         DbgPrint _x_ ; \
     }
@@ -34,8 +35,9 @@ extern ULONG DebugTraceLevel;
 #else /* _MSC_VER */
 
 #define NDIS_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d)(%s) ", __FILE__, __LINE__, __FUNCTION__); \
         DbgPrint _x_ ; \
     }
diff --git a/reactos/drivers/network/dd/rtl8139/debug.h b/reactos/drivers/network/dd/rtl8139/debug.h
index f495766..73e7189 100644 (file)
--- a/reactos/drivers/network/dd/rtl8139/debug.h
+++ b/reactos/drivers/network/dd/rtl8139/debug.h
@@ -25,8 +25,9 @@ extern ULONG DebugTraceLevel;
 #ifdef _MSC_VER
 
 #define NDIS_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d) ", __FILE__, __LINE__); \
         DbgPrint _x_ ; \
     }
@@ -34,8 +35,9 @@ extern ULONG DebugTraceLevel;
 #else /* _MSC_VER */
 
 #define NDIS_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d)(%s) ", __FILE__, __LINE__, __FUNCTION__); \
         DbgPrint _x_ ; \
     }
diff --git a/reactos/drivers/network/lan/include/debug.h b/reactos/drivers/network/lan/include/debug.h
index fc72311..5147292 100644 (file)
--- a/reactos/drivers/network/lan/include/debug.h
+++ b/reactos/drivers/network/lan/include/debug.h
@@ -40,8 +40,9 @@ extern DWORD DebugTraceLevel;
 #ifdef _MSC_VER
 
 #define LA_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d) ", __FILE__, __LINE__); \
         DbgPrint _x_ ; \
     }
@@ -49,8 +50,9 @@ extern DWORD DebugTraceLevel;
 #else /* _MSC_VER */
 
 #define LA_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d)(%s) ", __FILE__, __LINE__, __FUNCTION__); \
         DbgPrint _x_ ; \
     }
diff --git a/reactos/drivers/network/ndis/include/debug.h b/reactos/drivers/network/ndis/include/debug.h
index 32cf70f..678d9bd 100644 (file)
--- a/reactos/drivers/network/ndis/include/debug.h
+++ b/reactos/drivers/network/ndis/include/debug.h
@@ -26,8 +26,9 @@ extern ULONG DebugTraceLevel;
 #ifdef _MSC_VER
 
 #define NDIS_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d) ", __FILE__, __LINE__); \
         DbgPrint _x_ ; \
     }
@@ -35,8 +36,9 @@ extern ULONG DebugTraceLevel;
 #else /* _MSC_VER */
 
 #define NDIS_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d)(%s) ", __FILE__, __LINE__, __FUNCTION__); \
         DbgPrint _x_ ; \
     }
diff --git a/reactos/tools/cabman/cabinet.h b/reactos/tools/cabman/cabinet.h
index 4794d5e..1a2df3e 100644 (file)
--- a/reactos/tools/cabman/cabinet.h
+++ b/reactos/tools/cabman/cabinet.h
@@ -65,8 +65,9 @@ extern ULONG DebugTraceLevel;
 
 #undef DPRINT
 #define DPRINT(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         printf("(%s:%d)(%s) ", __FILE__, __LINE__, __FUNCTION__); \
         printf _x_ ; \
     }
diff --git a/rosapps/applications/net/tditest/include/debug.h b/rosapps/applications/net/tditest/include/debug.h
index 349b521..f4eb3b1 100644 (file)
--- a/rosapps/applications/net/tditest/include/debug.h
+++ b/rosapps/applications/net/tditest/include/debug.h
@@ -28,8 +28,9 @@ extern ULONG DebugTraceLevel;
 #ifdef _MSC_VER
 
 #define TDI_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d) ", __FILE__, __LINE__); \
         DbgPrint _x_ ; \
     }
@@ -37,8 +38,9 @@ extern ULONG DebugTraceLevel;
 #else /* _MSC_VER */
 
 #define TDI_DbgPrint(_t_, _x_) \
-    if (((DebugTraceLevel & NORMAL_MASK) >= _t_) || \
-        ((DebugTraceLevel & _t_) > NORMAL_MASK)) { \
+    if ((_t_ > NORMAL_MASK) \
+        ? (DebugTraceLevel & _t_) > NORMAL_MASK \
+        : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
         DbgPrint("(%s:%d)(%s) ", __FILE__, __LINE__, __FUNCTION__); \
         DbgPrint _x_ ; \
     }
A free Windows-compatible Operating System