[Lldb-commits] [lldb] r254546 - Make --results-file stdout implied if unspecified when using a results formatter.

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Wed Dec 2 13:12:17 PST 2015


Author: tfiala
Date: Wed Dec  2 15:12:17 2015
New Revision: 254546

URL: http://llvm.org/viewvc/llvm-project?rev=254546&view=rev
Log:
Make --results-file stdout implied if unspecified when using a results formatter.

Also cleans up pylint warnings (stock settings) in the modified function.

Modified:
    lldb/trunk/packages/Python/lldbsuite/test/dotest.py

Modified: lldb/trunk/packages/Python/lldbsuite/test/dotest.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/dotest.py?rev=254546&r1=254545&r2=254546&view=diff
==============================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/dotest.py (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/dotest.py Wed Dec  2 15:12:17 2015
@@ -935,7 +935,15 @@ def setupTestResults():
         # Connect to the specified localhost port.
         results_file_object, cleanup_func = createSocketToLocalPort(
             results_port)
-        default_formatter_name = "lldbsuite.test.test_results.RawPickledFormatter"
+        default_formatter_name = (
+            "lldbsuite.test.test_results.RawPickledFormatter")
+
+    # If we have a results formatter name specified and we didn't specify
+    # a results file, we should use stdout.
+    if results_formatter_name is not None and results_file_object is None:
+        # Use stdout.
+        results_file_object = sys.stdout
+        cleanup_func = None
 
     if results_file_object:
         # We care about the formatter.  Choose user-specified or, if
@@ -945,7 +953,8 @@ def setupTestResults():
         else:
             formatter_name = default_formatter_name
 
-        # Create an instance of the class.  First figure out the package/module.
+        # Create an instance of the class.
+        # First figure out the package/module.
         components = formatter_name.split(".")
         module = importlib.import_module(".".join(components[:-1]))
 
@@ -963,7 +972,8 @@ def setupTestResults():
             command_line_options)
 
         # Create the TestResultsFormatter given the processed options.
-        results_formatter_object = clazz(results_file_object, formatter_options)
+        results_formatter_object = clazz(
+            results_file_object, formatter_options)
 
         # Start the results formatter session - we'll only have one
         # during a given dotest process invocation.




More information about the lldb-commits mailing list