Skip to content

Commit 0b37c45

Browse files
committed
[lldb][ValueObject][NFC] Remove unused SyntheticChildrenFrontEnd member
These `IsValid`/`SetValid` APIs are only ever used from 1 data-formatter in the Swift LLDB fork. Since all the APIs on `SyntheticChildrenFrontEnd` are meant to be overriden, there is no good way to enforce calling `IsValid` from the base. And we should just let that 1 data-formatter manage its own `IsValid` state.
1 parent d13bcb6 commit 0b37c45

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

lldb/include/lldb/DataFormatters/TypeSynthetic.h

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,9 @@ class SyntheticChildrenFrontEnd {
2828
protected:
2929
ValueObject &m_backend;
3030

31-
void SetValid(bool valid) { m_valid = valid; }
32-
33-
bool IsValid() { return m_valid; }
34-
3531
public:
3632
SyntheticChildrenFrontEnd(ValueObject &backend)
37-
: m_backend(backend), m_valid(true) {}
33+
: m_backend(backend) {}
3834

3935
virtual ~SyntheticChildrenFrontEnd() = default;
4036

@@ -100,7 +96,6 @@ class SyntheticChildrenFrontEnd {
10096
CompilerType type);
10197

10298
private:
103-
bool m_valid;
10499
SyntheticChildrenFrontEnd(const SyntheticChildrenFrontEnd &) = delete;
105100
const SyntheticChildrenFrontEnd &
106101
operator=(const SyntheticChildrenFrontEnd &) = delete;

0 commit comments

Comments
 (0)