Add a comment about a more efficient implementation of btree range erase.
PiperOrigin-RevId: 561954737
Change-Id: I0e29a4f4523e4b3eafbd26d1d96f6e1c8deed957
diff --git a/absl/container/internal/btree.h b/absl/container/internal/btree.h
index 569faa0..5e8a252 100644
--- a/absl/container/internal/btree.h
+++ b/absl/container/internal/btree.h
@@ -2534,6 +2534,10 @@
return res;
}
+// Note: we tried implementing this more efficiently by erasing all of the
+// elements in [begin, end) at once and then doing rebalancing once at the end
+// (rather than interleaving deletion and rebalancing), but that adds a lot of
+// complexity, which seems to outweigh the performance win.
template <typename P>
auto btree<P>::erase_range(iterator begin, iterator end)
-> std::pair<size_type, iterator> {
