[Lldb-commits] [PATCH] D153921: [lldb] Assert index is valid in DWARFDeclContext

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere marked an inline comment as done.
JDevlieghere added inline comments.


================
Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.h:58
   Entry &operator[](uint32_t idx) {
-    // "idx" must be valid
+    assert(idx < m_entries.size() && "invalid index");
     return m_entries[idx];
----------------
fdeazeve wrote:
> Note the difference in how you used () between the two asserts
Yup, typo below. I put up the patch before my build finished :-) 


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

https://reviews.llvm.org/D153921