Fixing Youngjae's comment

Author: littlepig2013

rocksdb/rocksdb_db.cc

@@ -218,16 +218,16 @@ void RocksdbDB::Init() {
 }
 
 void RocksdbDB::Cleanup() { 
+  const std::lock_guard<std::mutex> lock(mu_);
+  if (--ref_cnt_) {
+    return;
+  }
   for (size_t i = 0; i < cf_handles_.size(); i++) {
     if (cf_handles_[i] != nullptr) {
       delete cf_handles_[i];
       cf_handles_[i] = nullptr;
     }
   }
-  const std::lock_guard<std::mutex> lock(mu_);
-  if (--ref_cnt_) {
-    return;
-  }
   delete db_;
 }
 

rocksdb/rocksdb_db.h

@@ -92,7 +92,7 @@ class RocksdbDB : public DB {
 
   int fieldcount_;
 
-  std::vector<rocksdb::ColumnFamilyHandle *> cf_handles_;
+  static std::vector<rocksdb::ColumnFamilyHandle *> cf_handles_;
   static rocksdb::DB *db_;
   static int ref_cnt_;
   static std::mutex mu_;