JDevlieghere added inline comments.
================
Comment at: lldb/examples/python/crashlog.py:1025
+ if not target:
+ result.PutCString("error: couldn't create target provided by the
user ({option.target_path})")
+ return
----------------
This is not an f-string so this will actually print `{option.target_path}`
rather than the actual value. That said, we don't use those anywhere else, so
let's use `.format()` to be consistent.
================
Comment at:
lldb/test/Shell/ScriptInterpreter/Python/Crashlog/scripted_crashlog_json.test:7
+# RUN: -o 'crashlog -a -i -t %t.dir/multithread-test
%S/Inputs/interactive_crashlog/multithread-test.ips' \
# RUN: -o "thread list" -o "bt all" 2>&1 | FileCheck %s
----------------
Can we test some of the error cases here? Should be easy enough to provide a
target that doesn't exist (and would've caught the f-string issue).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129611/new/
https://reviews.llvm.org/D129611
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
