From 02e62ebbc88891969d59fc315be7e0f112e56d89 Mon Sep 17 00:00:00 2001
From: Edouard A <edouard@quasardb.net>
Date: Wed, 16 Mar 2016 22:57:57 +0100
Subject: [PATCH] Fixes warnings and ensure correct int behavior on 32-bit
 platforms.

---
 db/managed_iterator.cc             | 5 +++--
 db/memtable_list.cc                | 4 ++--
 db/version_set.cc                  | 4 ++--
 table/block_based_table_factory.cc | 6 +++---
 util/arena.cc                      | 2 +-
 util/random.h                      | 2 +-
 util/testutil.cc                   | 2 +-
 7 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/db/managed_iterator.cc b/db/managed_iterator.cc
index ceb7ba40b..1d47f933d 100644
--- a/db/managed_iterator.cc
+++ b/db/managed_iterator.cc
@@ -79,7 +79,7 @@ ManagedIterator::ManagedIterator(DBImpl* db, const ReadOptions& read_options,
       release_supported_(true) {
   read_options_.managed = false;
   if ((!read_options_.tailing) && (read_options_.snapshot == nullptr)) {
-    assert(read_options_.snapshot = db_->GetSnapshot());
+    assert(nullptr != (read_options_.snapshot = db_->GetSnapshot()));
     snapshot_created_ = true;
   }
   cfh_.SetCFD(cfd);
@@ -210,7 +210,8 @@ void ManagedIterator::RebuildIterator() {
 void ManagedIterator::UpdateCurrent() {
   assert(mutable_iter_ != nullptr);
 
-  if (!(valid_ = mutable_iter_->Valid())) {
+  valid_ = mutable_iter_->Valid();
+  if (!valid_) {
     status_ = mutable_iter_->status();
     return;
   }
diff --git a/db/memtable_list.cc b/db/memtable_list.cc
index 9668f8396..9c1d3632b 100644
--- a/db/memtable_list.cc
+++ b/db/memtable_list.cc
@@ -345,8 +345,8 @@ Status MemTableList::InstallMemtableFlushResults(
         imm_flush_needed.store(true, std::memory_order_release);
       }
       ++mem_id;
-    } while (!current_->memlist_.empty() && (m = current_->memlist_.back()) &&
-             m->file_number_ == file_number);
+    } while (!current_->memlist_.empty() && (nullptr != (m = current_->memlist_.back())) &&
+             (m->file_number_ == file_number));
   }
   commit_in_progress_ = false;
   return s;
diff --git a/db/version_set.cc b/db/version_set.cc
index fd53a4adb..1da447540 100644
--- a/db/version_set.cc
+++ b/db/version_set.cc
@@ -90,8 +90,8 @@ class FilePicker {
              const Comparator* user_comparator,
              const InternalKeyComparator* internal_comparator)
       : num_levels_(num_levels),
-        curr_level_(-1),
-        hit_file_level_(-1),
+        curr_level_(static_cast<unsigned int>(-1)),
+        hit_file_level_(static_cast<unsigned int>(-1)),
         search_left_bound_(0),
         search_right_bound_(FileIndexer::kLevelMaxIndex),
 #ifndef NDEBUG
diff --git a/table/block_based_table_factory.cc b/table/block_based_table_factory.cc
index 4cfff93a3..75917232d 100644
--- a/table/block_based_table_factory.cc
+++ b/table/block_based_table_factory.cc
@@ -110,7 +110,7 @@ std::string BlockBasedTableFactory::GetPrintableTableOptions() const {
 
   snprintf(buffer, kBufferSize, "  flush_block_policy_factory: %s (%p)\n",
            table_options_.flush_block_policy_factory->Name(),
-           table_options_.flush_block_policy_factory.get());
+           static_cast<void*>(table_options_.flush_block_policy_factory.get()));
   ret.append(buffer);
   snprintf(buffer, kBufferSize, "  cache_index_and_filter_blocks: %d\n",
            table_options_.cache_index_and_filter_blocks);
@@ -128,7 +128,7 @@ std::string BlockBasedTableFactory::GetPrintableTableOptions() const {
            table_options_.no_block_cache);
   ret.append(buffer);
   snprintf(buffer, kBufferSize, "  block_cache: %p\n",
-           table_options_.block_cache.get());
+           static_cast<void*>(table_options_.block_cache.get()));
   ret.append(buffer);
   if (table_options_.block_cache) {
     snprintf(buffer, kBufferSize, "  block_cache_size: %" ROCKSDB_PRIszt "\n",
@@ -136,7 +136,7 @@ std::string BlockBasedTableFactory::GetPrintableTableOptions() const {
     ret.append(buffer);
   }
   snprintf(buffer, kBufferSize, "  block_cache_compressed: %p\n",
-           table_options_.block_cache_compressed.get());
+           static_cast<void*>(table_options_.block_cache_compressed.get()));
   ret.append(buffer);
   if (table_options_.block_cache_compressed) {
     snprintf(buffer, kBufferSize,
diff --git a/util/arena.cc b/util/arena.cc
index 8bb482dbf..77eec10e7 100644
--- a/util/arena.cc
+++ b/util/arena.cc
@@ -26,7 +26,7 @@ const size_t Arena::kInlineSize;
 #endif
 
 const size_t Arena::kMinBlockSize = 4096;
-const size_t Arena::kMaxBlockSize = 2 << 30;
+const size_t Arena::kMaxBlockSize = 2u << 30;
 static const int kAlignUnit = sizeof(void*);
 
 size_t OptimizeBlockSize(size_t block_size) {
diff --git a/util/random.h b/util/random.h
index 57d5bd65b..7428454d8 100644
--- a/util/random.h
+++ b/util/random.h
@@ -102,7 +102,7 @@ class Random64 {
   // return "base" random bits.  The effect is to pick a number in the
   // range [0,2^max_log-1] with exponential bias towards smaller numbers.
   uint64_t Skewed(int max_log) {
-    return Uniform(1 << Uniform(max_log + 1));
+    return Uniform(uint64_t(1) << Uniform(max_log + 1));
   }
 };
 
diff --git a/util/testutil.cc b/util/testutil.cc
index 527402706..b8190faf7 100644
--- a/util/testutil.cc
+++ b/util/testutil.cc
@@ -113,7 +113,7 @@ class Uint64ComparatorImpl : public Comparator {
 };
 }  // namespace
 
-static port::OnceType once = LEVELDB_ONCE_INIT;
+static port::OnceType once;
 static const Comparator* uint64comp;
 
 static void InitModule() {