[Lldb-commits] [PATCH] D97287: [lldb][NFC] Rename the second ValueObjectManager to ValueObjectUpdater and remove the dead code

[Raphael Isemann via Phabricator via lldb-commits]

teemperor created this revision.
teemperor added a reviewer: LLDB.
teemperor added a project: LLDB.
Herald added subscribers: JDevlieghere, mgorny.
teemperor requested review of this revision.

`ValueObject.h` contains the `ValueObject::ValueObjectManager` type which is just a typedef
for the ClusterManager that takes care of the whole ValueObject memory management. However,
there is also `ValueObjectManager` defined in the same header which is only used in the curses UI
implementation and consists mostly of dead and completely untested code.

This code been around since a while (it was added in 2016 as 8369b28da0750129ababae357bea98940800a0e0),
so I think we shouldn't just revert the whole patch.

Instead this patch just moves the class to its own header that it isn't just hiding in the ValueObject
header and renames it to `ValueObjectUpdater` that it at least has a unique name (which I hope also slightly
better reflects the purpose of this class). I also deleted all the dead code branches and functions.


https://reviews.llvm.org/D97287

Files:
  lldb/include/lldb/Core/ValueObject.h
  lldb/include/lldb/Core/ValueObjectUpdater.h
  lldb/source/Core/CMakeLists.txt
  lldb/source/Core/IOHandlerCursesGUI.cpp
  lldb/source/Core/ValueObject.cpp
  lldb/source/Core/ValueObjectUpdater.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97287.325773.patch
Type: text/x-patch
Size: 10414 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20210223/dfd02203/attachment.bin>