================
@@ -450,6 +450,8 @@ class RangeDataVector {
 
   void Append(const Entry &entry) { m_entries.emplace_back(entry); }
 
+  void Append(B &&b, S &&s, T &&t) { m_entries.emplace_back(Entry(b, s, t)); }
----------------
mbucko wrote:

why not just name then base, size, data? Then the reader doesn't have to read 
the function contract. Also, would be curious what the guideline says about 
single letter naming for variables.

https://github.com/llvm/llvm-project/pull/107937
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to