LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

[rfc 1/2] [PATCH 1/2] Slightly simplify options conflicts logic

To: lvs-devel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, netfilter@xxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxx
Subject: [rfc 1/2] [PATCH 1/2] Slightly simplify options conflicts logic
Cc: Wensong Zhang <wensong@xxxxxxxxxxxx>, Julian Anastasov <ja@xxxxxx>, Patrick McHardy <kaber@xxxxxxxxx>
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Thu, 05 Aug 2010 20:47:56 +0900
Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx>
---
 ipvsadm.c |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/ipvsadm.c b/ipvsadm.c
index 76ec7c4..1ac6c7f 100644
--- a/ipvsadm.c
+++ b/ipvsadm.c
@@ -763,11 +763,9 @@ static int process_options(int argc, char **argv, int 
reading_stdin)
 
        switch (ce.cmd) {
        case CMD_LIST:
-               if ((options & OPT_CONNECTION ||
-                    options & OPT_TIMEOUT || options & OPT_DAEMON) &&
-                   (options & OPT_STATS ||
-                    options & OPT_PERSISTENTCONN ||
-                    options & OPT_RATE || options & OPT_THRESHOLDS))
+               if (options & (OPT_CONNECTION|OPT_TIMEOUT|OPT_DAEMON) &&
+                   options & (OPT_STATS|OPT_PERSISTENTCONN|
+                              OPT_RATE|OPT_THRESHOLDS))
                        fail(2, "options conflicts in the list command");
 
                if (options & OPT_CONNECTION)
-- 
1.7.1


--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>