From 38d54832f70c3228ad975acec5688fb410678c1a Mon Sep 17 00:00:00 2001 From: Mayank Agarwal Date: Thu, 21 Mar 2013 11:12:42 -0700 Subject: [PATCH] Initialize variable in constructor for PosixEnv::checkedDiskForMmap_ Summary: This caused compilation problems on some gcc platforms during the third-partyrelease Test Plan: make Reviewers: sheki Reviewed By: sheki Differential Revision: https://reviews.facebook.net/D9627 --- util/env_posix.cc | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/util/env_posix.cc b/util/env_posix.cc index 3e9f1b529..e2633f02c 100644 --- a/util/env_posix.cc +++ b/util/env_posix.cc @@ -907,8 +907,8 @@ class PosixEnv : public Env { } private: - bool checkedDiskForMmap_ = false; - bool forceMmapOff = false; // do we override Env options? + bool checkedDiskForMmap_; + bool forceMmapOff; // do we override Env options? void PthreadCall(const char* label, int result) { if (result != 0) { @@ -965,7 +965,9 @@ class PosixEnv : public Env { BGQueue queue_; }; -PosixEnv::PosixEnv() : page_size_(getpagesize()), +PosixEnv::PosixEnv() : checkedDiskForMmap_(false), + forceMmapOff(false), + page_size_(getpagesize()), started_bgthread_(0), num_threads_(1), queue_size_(0) {