[dm-devel] tiny patch: avoid an unnecessary test

Jim Meyering jim at meyering.net
Fri Mar 16 08:28:43 UTC 2007


Not a big deal (barely worth mentioning), but it might save
someone else a little time wondering "why".

2007-03-11  Jim Meyering  <jim at meyering.net>

	Remove unnecessary test.
	* dmeventd/dmeventd.c (_alloc_dso_data): Don't test known-non-NULL
	memset return value.

2007-03-11  Jim Meyering  <jim at meyering.net>
Index: dmeventd/dmeventd.c
===================================================================
RCS file: /cvs/dm/device-mapper/dmeventd/dmeventd.c,v
retrieving revision 1.46
diff -u -p -r1.46 dmeventd.c
--- dmeventd/dmeventd.c	2 Feb 2007 17:08:51 -0000	1.46
+++ dmeventd/dmeventd.c	12 Mar 2007 14:39:35 -0000
@@ -258,8 +258,8 @@ static struct dso_data *_alloc_dso_data(
 	if (!ret)
 		return NULL;

-	if (!memset(ret, 0, sizeof(*ret)) ||
-	    !(ret->dso_name = dm_strdup(data->dso_name))) {
+	memset(ret, 0, sizeof(*ret));
+	if (!(ret->dso_name = dm_strdup(data->dso_name))) {
 		dm_free(ret);
 		return NULL;
 	}




More information about the dm-devel mailing list