[Lldb-commits] [PATCH] D68305: [JSON] Remove Utility/JSON.{h|cpp}

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere created this revision.
JDevlieghere added reviewers: labath, vsk, aprantl, clayborg.
Herald added a subscriber: mgorny.
Herald added a project: LLDB.
JDevlieghere retitled this revision from "[JSON] RemoveUtility/JSON.{h|cpp}" to "[JSON] Remove Utility/JSON.{h|cpp}".

This patch is the final step in my quest to get rid of the JSON parser in LLDB. Vedant's coverage report [1] shows that it was mostly untested. Furthermore, the LLVM implementation has a much nicer API and using it means one less thing to maintain for LLDB.

[1] http://lab.llvm.org:8080/coverage/coverage-reports/index.html


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D68305

Files:
  lldb/include/lldb/Utility/JSON.h
  lldb/source/Utility/CMakeLists.txt
  lldb/source/Utility/JSON.cpp
  lldb/unittests/Utility/CMakeLists.txt
  lldb/unittests/Utility/JSONTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68305.222716.patch
Type: text/x-patch
Size: 23510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20191001/c3242336/attachment-0001.bin>