[lvm-devel] master - pool: drop duplicated sign test

Zdenek Kabelac zkabelac at sourceware.org
Fri Mar 10 18:35:13 UTC 2017


Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=8c72878cf2f53811cf5ae6fc9b7c787feb74e0c6
Commit:        8c72878cf2f53811cf5ae6fc9b7c787feb74e0c6
Parent:        acfc82ae29eea2b107fb97e4d6fd350b3347f925
Author:        Zdenek Kabelac <zkabelac at redhat.com>
AuthorDate:    Thu Mar 9 17:09:31 2017 +0100
Committer:     Zdenek Kabelac <zkabelac at redhat.com>
CommitterDate: Fri Mar 10 19:33:00 2017 +0100

pool: drop duplicated sign test

Avoid duplicate testing of '-' sign.
Already checked with arg_from_list_is_negative().
---
 tools/toollib.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/tools/toollib.c b/tools/toollib.c
index f2951a7..c10f5ca 100644
--- a/tools/toollib.c
+++ b/tools/toollib.c
@@ -1243,11 +1243,6 @@ int get_pool_params(struct cmd_context *cmd,
 	}
 
 	if (arg_is_set(cmd, poolmetadatasize_ARG)) {
-		if (arg_sign_value(cmd, poolmetadatasize_ARG, SIGN_NONE) == SIGN_MINUS) {
-			log_error("Negative pool metadata size is invalid.");
-			return 0;
-		}
-
 		if (arg_is_set(cmd, poolmetadata_ARG)) {
 			log_error("Please specify either metadata logical volume or its size.");
 			return 0;




More information about the lvm-devel mailing list