master - cleanup: simplier formula

Zdenek Kabelac <[email protected]>
Newsgroups dev.linux.lists.lvm-devel
Message-ID <[email protected]>
Gitweb:        http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=4310dfd4e141bb83320148d245f3ab2cae778f6e
Commit:        4310dfd4e141bb83320148d245f3ab2cae778f6e
Parent:        ebcfd09ba9631b52762b4932fbe06260326b9827
Author:        Zdenek Kabelac <[email protected]>
AuthorDate:    Fri Jan 8 11:07:47 2016 +0100
Committer:     Zdenek Kabelac <[email protected]>
CommitterDate: Thu Jan 14 11:34:05 2016 +0100

cleanup: simplier formula

---
 lib/metadata/lv_manip.c |   21 +++++++--------------
 1 files changed, 7 insertions(+), 14 deletions(-)

diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c
index 62564c5..1396b86 100644
--- a/lib/metadata/lv_manip.c
+++ b/lib/metadata/lv_manip.c
@@ -4950,21 +4950,14 @@ static int _lvresize_adjust_extents(struct cmd_context *cmd, struct logical_volu
 			log_print_unless_silent("Ignoring stripes, stripesize and mirrors "
 						"arguments when reducing.");
 
-		if (lp->sign == SIGN_MINUS) 
-			if (lp->extents_are_pes) {
-				if (lp->extents >= existing_physical_extents) {
-					log_error("Unable to reduce %s below 1 extent.", lp->lv_name);
-					return 0;
-				}
-				new_extents = existing_physical_extents - lp->extents;
-			} else {
-				new_extents = existing_logical_extents - lp->extents;
-				if (lp->extents >= existing_logical_extents) {
-					log_error("Unable to reduce %s below 1 extent.", lp->lv_name);
-					return 0;
-				}
+		if (lp->sign == SIGN_MINUS)  {
+			new_extents = lp->extents_are_pes ? existing_physical_extents : existing_logical_extents;
+			if (lp->extents >= new_extents) {
+				log_error("Unable to reduce %s below 1 extent.", lp->lv_name);
+				return 0;
 			}
-		else
+			new_extents -= lp->extents;
+		} else
 			new_extents = lp->extents;
 
 		dm_list_iterate_items(seg, &lv->segments) {
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.