stream.py: Make usage() function return a string.

This will marginally simplify a future patch.

diff --git a/python/ovs/stream.py b/python/ovs/stream.py
index 5180642dead01861504fb024e2779d28887aa5d4..3bb0c10fd2b165957b0e320af5c510464ca50e97 100644 (file)
--- a/python/ovs/stream.py
+++ b/python/ovs/stream.py
@@ -302,14 +302,10 @@ class PassiveStream(object):
         self.socket.close()
 
 
-def usage(name, active, passive):
-    print
-    if active:
-        print("Active %s connection methods:" % name)
-        print("  unix:FILE               "
-               "Unix domain socket named FILE")
-
-    if passive:
-        print("Passive %s connection methods:" % name)
-        print("  punix:FILE              "
-              "listen on Unix domain socket FILE")
+def usage(name):
+    return """
+Active %s connection methods:
+  unix:FILE               Unix domain socket named FILE
+
+Passive %s connection methods:
+  punix:FILE              Listen on Unix domain socket FILE""" % (name, name)

diff --git a/tests/test-jsonrpc.py b/tests/test-jsonrpc.py
index 9da3bd2d73ce5a32d086e82924af31885cab552e..5ce45ad2f3a9a7f476d4c14d9b200f18a6d6e823 100644 (file)
--- a/tests/test-jsonrpc.py
+++ b/tests/test-jsonrpc.py
@@ -212,7 +212,7 @@ usage: %s [OPTIONS] COMMAND [ARG...]
   request REMOTE METHOD PARAMS   send request, print reply
   notify REMOTE METHOD PARAMS  send notification and exit
 """ % (ovs.util.PROGRAM_NAME, ovs.util.PROGRAM_NAME))
-    ovs.stream.usage("JSON-RPC", True, True)
+    sys.stdout.write(ovs.stream.usage("JSON-RPC") + "\n")
     ovs.daemon.usage()
     sys.stdout.write("""
 Other options: