[LNT] r311204 - This default arg was incorrect

Chris Matthews via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 18 14:43:48 PDT 2017


Author: cmatthews
Date: Fri Aug 18 14:43:48 2017
New Revision: 311204

URL: http://llvm.org/viewvc/llvm-project?rev=311204&view=rev
Log:
This default arg was incorrect

Default args are only evaluated at program start, so this would be the
wrong timestamp.

Modified:
    lnt/trunk/lnt/tests/builtintest.py

Modified: lnt/trunk/lnt/tests/builtintest.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/tests/builtintest.py?rev=311204&r1=311203&r2=311204&view=diff
==============================================================================
--- lnt/trunk/lnt/tests/builtintest.py (original)
+++ lnt/trunk/lnt/tests/builtintest.py Fri Aug 18 14:43:48 2017
@@ -42,7 +42,9 @@ class BuiltinTest(object):
         """
         raise RuntimeError("Abstract Method.")
 
-    def log(self, message, ts=timestamp()):
+    def log(self, message, ts=None):
+        if not ts:
+            ts = timestamp()
         print >>sys.stderr, '%s: %s' % (ts, message)
 
     @staticmethod




More information about the llvm-commits mailing list