From 2e1ad2c48b60aafe9a030f206456321ee04bb030 Mon Sep 17 00:00:00 2001 From: Abhishek Kona Date: Tue, 8 Jan 2013 15:55:06 -0800 Subject: [PATCH] Remove unnecessary asserts in table/merger.cc Summary: The asserts introduced in https://reviews.facebook.net/D7629 are wrong. The direction of iteration is changed after the function call so they assert's fail. Test Plan: make clean check Reviewers: dhruba Reviewed By: dhruba CC: leveldb Differential Revision: https://reviews.facebook.net/D7827 --- table/merger.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/table/merger.cc b/table/merger.cc index 5f7b24bd8..7c5ef92f4 100644 --- a/table/merger.cc +++ b/table/merger.cc @@ -188,7 +188,6 @@ class MergingIterator : public Iterator { }; void MergingIterator::FindSmallest() { - assert (direction_ == kForward); if (minHeap_.empty()) { current_ = NULL; } else { @@ -199,7 +198,6 @@ void MergingIterator::FindSmallest() { } void MergingIterator::FindLargest() { - assert(direction_ == kReverse); if (maxHeap_.empty()) { current_ = NULL; } else {