[lvm-devel] [PATCH 23/23] Remove dead assignment in vg_split_mdas

Zdenek Kabelac zkabelac at redhat.com
Tue Dec 21 15:41:55 UTC 2010


'common_mda' is assigned again with next code line.

Why is the result not important from the first call?
CHECKME!

Signed-off-by: Zdenek Kabelac <zkabelac at redhat.com>
---
 lib/metadata/metadata.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/lib/metadata/metadata.c b/lib/metadata/metadata.c
index 27abaea..ebd19fa 100644
--- a/lib/metadata/metadata.c
+++ b/lib/metadata/metadata.c
@@ -1263,15 +1263,14 @@ int vg_split_mdas(struct cmd_context *cmd __attribute__((unused)),
 {
 	struct dm_list *mdas_from_in_use, *mdas_to_in_use;
 	struct dm_list *mdas_from_ignored, *mdas_to_ignored;
-	int common_mda = 0;
+	int common_mda;
 
 	mdas_from_in_use = &vg_from->fid->metadata_areas_in_use;
 	mdas_from_ignored = &vg_from->fid->metadata_areas_ignored;
 	mdas_to_in_use = &vg_to->fid->metadata_areas_in_use;
 	mdas_to_ignored = &vg_to->fid->metadata_areas_ignored;
 
-	common_mda = _move_mdas(vg_from, vg_to,
-				mdas_from_in_use, mdas_to_in_use);
+	_move_mdas(vg_from, vg_to, mdas_from_in_use, mdas_to_in_use);
 	common_mda = _move_mdas(vg_from, vg_to,
 				mdas_from_ignored, mdas_to_ignored);
 
-- 
1.7.3.4




More information about the lvm-devel mailing list