From 39c14891b695fc02fda5b45d77ddfd0f80c41816 Mon Sep 17 00:00:00 2001 From: Kai Liu Date: Sun, 27 Oct 2013 22:55:46 -0700 Subject: [PATCH] Fix the gcc warning for unused variable Summary: Fix the unused variable warning for `first` when running `make release` Test Plan: make make check Reviewers: dhruba, igor CC: leveldb Differential Revision: https://reviews.facebook.net/D13695 --- db/memtablelist.cc | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/db/memtablelist.cc b/db/memtablelist.cc index 5f7c7c5031..ca9671852b 100644 --- a/db/memtablelist.cc +++ b/db/memtablelist.cc @@ -96,14 +96,12 @@ Status MemTableList::InstallMemtableFlushResults( } // flush was sucessful - bool first = true; - for (MemTable* m : mems) { + for (size_t i = 0; i < mems.size(); ++i) { // All the edits are associated with the first memtable of this batch. - assert(first || m->GetEdits()->NumEntries() == 0); - first = false; + assert(i == 0 || m->GetEdits()->NumEntries() == 0); - m->flush_completed_ = true; - m->file_number_ = file_number; + mems[i]->flush_completed_ = true; + mems[i]->file_number_ = file_number; } // if some other thread is already commiting, then return