On Mon, 24 Sep 2012, Marcin Cieslak wrote:

> NULL != NULL makes no sense, really...

heh, a comedy of cut/paste errors...

> ---
> cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C |    2 +-
> cde/programs/dtmail/dtmail/CheckBoxUiItem.C     |    2 +-
> cde/programs/dtmail/dtmail/ComposeCmds.C        |    2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C 
> b/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C
> index 0b3c8e5..0a70667 100644
> --- a/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C
> +++ b/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C
> @@ -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);
> }
>
> diff --git a/cde/programs/dtmail/dtmail/CheckBoxUiItem.C 
> b/cde/programs/dtmail/dtmail/CheckBoxUiItem.C
> index c2651b4..468a41b 100644
> --- a/cde/programs/dtmail/dtmail/CheckBoxUiItem.C
> +++ b/cde/programs/dtmail/dtmail/CheckBoxUiItem.C
> @@ -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);
> }
>
> diff --git a/cde/programs/dtmail/dtmail/ComposeCmds.C 
> b/cde/programs/dtmail/dtmail/ComposeCmds.C
> index d3117b3..0a6d695 100644
> --- a/cde/programs/dtmail/dtmail/ComposeCmds.C
> +++ b/cde/programs/dtmail/dtmail/ComposeCmds.C
> @@ -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();
>

-- 
Jon Trulson

The Higgs Field is what make atoms matter.
                 -- Tom L.



------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to