[Lldb-commits] [PATCH] D88264: [intel-pt] Refactor the JSON parsing
walter erquinigo via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Sep 24 15:15:52 PDT 2020
wallace created this revision.
Herald added a reviewer: JDevlieghere.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
wallace requested review of this revision.
Recently https://reviews.llvm.org/D88103 introduced a nice API for
converting a JSON object into C++ types, which include nice error
messaging.
I'm using that new functioniality to perform the parsing in a much more
elegant way. As a result, the code looks simpler and more maintainable,
as we aren't parsing anymore individual fields manually.
I updated the test cases accordingly.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D88264
Files:
lldb/include/lldb/Target/Trace.h
lldb/include/lldb/Target/TraceSettingsParser.h
lldb/source/Plugins/Trace/intel-pt/TraceIntelPTSettingsParser.cpp
lldb/source/Plugins/Trace/intel-pt/TraceIntelPTSettingsParser.h
lldb/source/Target/Trace.cpp
lldb/source/Target/TraceSettingsParser.cpp
lldb/test/API/commands/trace/TestTraceLoad.py
lldb/test/API/commands/trace/intelpt-trace/trace_bad4.json
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88264.294185.patch
Type: text/x-patch
Size: 30903 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200924/88d8418a/attachment-0001.bin>
More information about the lldb-commits
mailing list