From f0c52912f0b69ba3d5b1e5157f3575957eebfb34 Mon Sep 17 00:00:00 2001
From: Zdenek Kabelac <zkabelac@redhat.com>
Date: Thu, 10 Nov 2011 12:43:05 +0000
Subject: [PATCH] Small comment and indent updates

---
 lib/metadata/lv_manip.c   | 1 -
 lib/metadata/thin_manip.c | 2 +-
 tools/lvcreate.c          | 5 +++--
 tools/toollib.c           | 1 +
 4 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c
index 04ebfc87d..4f7104d2a 100644
--- a/lib/metadata/lv_manip.c
+++ b/lib/metadata/lv_manip.c
@@ -4263,7 +4263,6 @@ static struct logical_volume *_lv_create_an_lv(struct volume_group *vg, struct l
 	init_dmeventd_monitor(lp->activation_monitoring);
 
 	if (seg_is_thin(lp)) {
-
 		/* For thin snapshot suspend active thin origin first */
 		if (org && lv_is_active(org)) {
 			if (!suspend_lv(cmd, org)) {
diff --git a/lib/metadata/thin_manip.c b/lib/metadata/thin_manip.c
index 7a2c160b3..56e50955e 100644
--- a/lib/metadata/thin_manip.c
+++ b/lib/metadata/thin_manip.c
@@ -346,7 +346,7 @@ int update_pool_lv(struct logical_volume *lv, int activate)
 	}
 
 	if (activate) {
-		/* If the pool was not yet activated, do it */
+		/* If the pool is not active, do activate deactivate */
 		if (!lv_is_active(lv)) {
 			if (!activate_lv_excl(lv->vg->cmd, lv))
 				return_0;
diff --git a/tools/lvcreate.c b/tools/lvcreate.c
index 9c6150cef..d079c1c9a 100644
--- a/tools/lvcreate.c
+++ b/tools/lvcreate.c
@@ -200,7 +200,8 @@ static int _determine_snapshot_type(struct volume_group *vg,
 	struct lv_list *lvl;
 
 	if (!(lvl = find_lv_in_vg(vg, lp->origin))) {
-		log_error("Snapshot origin LV %s not found in Volume group %s.", lp->origin, vg->name);
+		log_error("Snapshot origin LV %s not found in Volume group %s.",
+			  lp->origin, vg->name);
 		return 0;
 	}
 
@@ -222,7 +223,7 @@ static int _determine_snapshot_type(struct volume_group *vg,
 
 /*
  * Update extents parameters based on other parameters which affect the size
- * calcuation.
+ * calculation.
  * NOTE: We must do this here because of the percent_t typedef and because we
  * need the vg.
  */
diff --git a/tools/toollib.c b/tools/toollib.c
index aba52dc2d..b9c4d6a65 100644
--- a/tools/toollib.c
+++ b/tools/toollib.c
@@ -329,6 +329,7 @@ int process_each_lv(struct cmd_context *cmd, int argc, char **argv,
 		dm_list_iterate_items(sll, &arg_lvnames) {
 			const char *vg_name = sll->str;
 			const char *lv_name = strchr(vg_name, '/');
+log_error("VGNAME strdup %p %s   %s", lv_name, vg_name,  vgname);
 
 			if ((!lv_name && !strcmp(vg_name, vgname))) {
 				/* Process all LVs in this VG */