[Lldb-commits] [PATCH] D54454: Be more permissive in what we consider a variable name.

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

Those seem legit things to try to capture, though a little esoteric.  Since "frame variable" and "target variable" didn't support these constructs before you should certainly add some tests for that.

The frame variable parser also supports:

(lldb) frame variable foo[0]

where foo is anything that can produce "vector" like children (e.g. std::vector's).  Will your change work with that?


https://reviews.llvm.org/D54454