[Lldb-commits] [PATCH] D74759: Treat RangeDataVector as an augmented BST

Thu Feb 27 08:54:40 PST 2020

labath added a comment.

In D74759#1893485 <https://reviews.llvm.org/D74759#1893485>, @unnar wrote:

> That should work...although I'm not sure how that is any different to the range or data being public. If a user modifies anything then it has essentially invalidated the whole structure anyway.

That is a fair point. I suppose the reason why I see this as different is because this field is an implementation detail of the RangeDataVector class, and so the user should not even see it -- whereas the user has a legitimate reason to at least access the other fields (and most of the methods only provide read-only access to these fields).

I'm sorry, I haven't gotten around to looking at this patch today, but I thought I'd at least say that...

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

https://reviews.llvm.org/D74759