[llvm][utils] Fix innocuous off by one in lldb formatters
authorDave Lee <davelee.com@gmail.com>
Fri, 26 Feb 2021 16:04:53 +0000 (08:04 -0800)
committerDave Lee <davelee.com@gmail.com>
Fri, 26 Feb 2021 16:10:41 +0000 (08:10 -0800)
num_children is "last_index" + 1, thus
    num_children + 1 = "last_index" + 2

this worked anyway because the index of `$$dereference$$` would work as long as
it was past the last index.

llvm/utils/lldbDataFormatters.py

index b759ee0..4dff5b0 100644 (file)
@@ -132,7 +132,7 @@ class OptionalSynthProvider:
 
     def get_child_index(self, name):
         if name == '$$dereference$$':
-            return self.valobj.num_children + 1
+            return self.valobj.num_children
         return self.valobj.GetIndexOfChildWithName(name)
 
     def get_child_at_index(self, index):