[Lldb-commits] [PATCH] D80448: [lldb/Test] Add a trace method to replace (commented out) print statements.

Martin Böhme via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jun 2 23:58:06 PDT 2020


mboehme added inline comments.


================
Comment at: lldb/packages/Python/lldbsuite/test/lldbtest.py:510
+        with recording(self, self.TraceOn()) as sbuf:
+            print(args, kwargs, file=sbuf)
+
----------------
labath wrote:
> JDevlieghere wrote:
> > This should be 
> > ```
> > print(*args, kwargs, file=sbuf)
> > ```
> Actually, it should be `print(*args, **kwargs, file=sbuf)`
With both Python 2 and 3, this, is giving me an error:

```
    print(*args, **kwargs, file=sbuf)
                         ^
SyntaxError: invalid syntax
```

I believe this should be `print(*args, file=sbuf, **kwargs)`. Can you fix this?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80448/new/

https://reviews.llvm.org/D80448





More information about the lldb-commits mailing list