[Lldb-commits] [PATCH] D131719: [lldb/crashlog] Adapt raw text crashlog exception to json format

Dave Lee via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Aug 11 15:22:24 PDT 2022


kastiglione added inline comments.


================
Comment at: lldb/examples/python/crashlog.py:708-709
+            if exception_extra_match:
+                if not self.crashlog.exception['type']:
+                    return
+                self.crashlog.exception['message'] = exception_extra_match.group(1)
----------------
is this meaning to check that the key `'type'` exists, and that it is false-y? Or, should it be checking whether the key `'type'` is in the dictionary (`if key is not in dict`)

also, this check could be hoisted out, and avoid performing the regex search if the result isn't going to be used. As it is now, the regex search happens, and the result is ignored if this condition is met.


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

https://reviews.llvm.org/D131719



More information about the lldb-commits mailing list