dtmail: fix warning: NULL used in arithmetic
authorMarcin Cieslak <saper@saper.info>
Mon, 24 Sep 2012 11:36:16 +0000 (13:36 +0200)
committerJon Trulson <jon@radscan.com>
Tue, 25 Sep 2012 00:27:18 +0000 (18:27 -0600)
NULL != NULL makes no sense, really...

cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C
cde/programs/dtmail/dtmail/CheckBoxUiItem.C
cde/programs/dtmail/dtmail/ComposeCmds.C

index 0b3c8e5a70816e3c1469d6401559390ef150c899..0a7066776a11e5565d8d3e1e09418902363af8fc 100644 (file)
@@ -99,7 +99,7 @@ void AntiCheckBoxUiItem::writeFromSourceToUi()
   else if (value == NULL || strcmp(value, "") == 0)
     options_checkbox_set_value(w, FALSE, this->dirty_bit);
   
-  if (NULL != NULL)
+  if (value != NULL)
     free((void*) value);
 }
 
index c2651b422ef34d29cba488fd45fd315804f12be1..468a41bc783cdcd956a2557cb01238bee8b937f2 100644 (file)
@@ -97,7 +97,7 @@ void CheckBoxUiItem::writeFromSourceToUi()
   else if (NULL == value || strcmp(value, "f") == 0)
     options_checkbox_set_value(w, FALSE, this->dirty_bit);
   
-  if (NULL != NULL)
+  if (value != NULL)
     free((void*) value);
 }
 
index d3117b3599076df01ec1d207c818f2c989f53702..0a6d695d0167be07af09639acf7ed07db2dd8c3f 100644 (file)
@@ -279,7 +279,7 @@ ComposeFamily::appendSignature(SendMsgDialog * compose)
     compose->get_editor()->textEditor()->append_to_contents(fullpath);
     if (NULL != fullpath)
       free((void*) fullpath);
-    if (NULL != NULL)
+    if (NULL != value)
       free((void*) value);
 
     compose->get_editor()->textEditor()->set_to_top();