[Lldb-commits] [lldb] [lldb] Add lookup by name to SBValue through new member property (PR #118814)

[Dave Lee via lldb-commits]

================
@@ -23,6 +23,13 @@ STRING_EXTENSION_OUTSIDE(SBValue)
                     if -count <= key < count:
                         key %= count
                         return self.sbvalue.GetChildAtIndex(key)
+                elif isinstance(key, str):
+                    if child := self.sbvalue.GetChildMemberWithName(key):
+                        return child
+                    # Support base classes, which are children but not members.
+                    for child in self.sbvalue:
+                        if child.name == key:
+                            return child
----------------
kastiglione wrote:

I've updated the PR to leave `child` alone, and introduced `member`. Does this work for everyone? It doesn't provide any means for accessing base classes by name, but that was not my primary intention.

https://github.com/llvm/llvm-project/pull/118814