Re: [evolution-patches] simple oops fix for mailer

[Not Zed <notzed ximian com>]

makes no difference of course.  both NULL and FALSE are merely conveniences to the actual c language definition of 0 for both values.

but for clarity, sure.

On Fri, 2004-08-06 at 13:24 -0400, Jeffrey Stedfast wrote:

bleh.

Jeff

text/plain attachment (oops.patch)

Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution/mail/ChangeLog,v
retrieving revision 1.3428
diff -u -r1.3428 ChangeLog
--- ChangeLog	4 Aug 2004 15:20:57 -0000	1.3428
+++ ChangeLog	6 Aug 2004 17:15:54 -0000
@@ -1,3 +1,8 @@
+2004-08-06  Jeffrey Stedfast  <fejj novell com>
+
+	* em-folder-tree-model.c (em_folder_tree_model_get_selected):
+	Return NULL not FALSE on error.
+
 2004-08-03  Jeffrey Stedfast  <fejj novell com>
 
 	* em-composer-utils.c (format_sender): If type="{Sender}" and name
Index: em-folder-tree-model.c
===================================================================
RCS file: /cvs/gnome/evolution/mail/em-folder-tree-model.c,v
retrieving revision 1.59
diff -u -r1.59 em-folder-tree-model.c
--- em-folder-tree-model.c	29 Jun 2004 10:05:54 -0000	1.59
+++ em-folder-tree-model.c	6 Aug 2004 17:15:54 -0000
@@ -1112,7 +1112,7 @@
 	
 	node = model->state ? model->state->children : NULL;
 	if (!node || strcmp (node->name, "tree-state") != 0)
-		return FALSE;
+		return NULL;
 	
 	node = node->children;
 	while (node != NULL) {

--

Michael Zucchi <notzed ximian com> "born to die, live to work, it's all downhill from here" Novell's Evolution and Free Software Developer