wdt cont build with delays

Summary: wdt cont build with delays Reviewed By: @nikunjy Differential Revision: D2501860 fb-gh-sync-id: 4fc85e79d1085d7a9a517a7c046ce82f0ebddc64
facebook · Oct 6, 2015 · 68d0512 · 68d0512
1 parent 56d8e5a
commit 68d0512
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -44,6 +44,8 @@ set(CMAKE_INSTALL_RPATH "${CMAKE_INSTALL_PREFIX}/lib")
 #set(CMAKE_BUILD_TYPE Debug)
 set(CMAKE_BUILD_TYPE Release)
 set(CMAKE_CXX_FLAGS "-msse4.2")
+#set(CMAKE_CXX_FLAGS "-msse4.2 -Wextra -Wsign-compare -Wunused-variable")
+#set(CMAKE_CXX_FLAGS "-msse4.2 -Wextra -Wsign-compare -Wunused-variable -Wconversion -Wsign-conversion")
 set(CMAKE_RUNTIME_OUTPUT_DIRECTORY "_bin/wdt")
 
 # Check that we have the Folly source tree

diff --git a/FileWriter.cpp b/FileWriter.cpp
@@ -31,7 +31,8 @@ ErrorCode FileWriter::open() {
     const int64_t ret = lseek(fd_, blockDetails_->offset, SEEK_SET);
     RECORD_PERF_RESULT(PerfStatReport::FILE_SEEK);
     if (ret < 0) {
-      PLOG(ERROR) << "Unable to seek " << blockDetails_->fileName;
+      PLOG(ERROR) << "Unable to seek to " << blockDetails_->offset << " for "
+                  << blockDetails_->fileName;
       close();
     }
   }