Remove warning, and add -Wextra

peadar · Sep 30, 2023 · 22cc95e · 22cc95e
1 parent 698f3e6
commit 22cc95e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -46,7 +46,7 @@ add_definitions(-DVERSION=${VERSION_TAG})
 set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_EXTENSIONS ON)
 
-add_definitions("-Wall -Wextra -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE")
+add_definitions("-Wall -Wmove -Wextra -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE")
 # Adding frame pointers makes things easy to run performance measurements with,
 # and doesn't cost much itself.
 add_definitions("-fno-omit-frame-pointer")

diff --git a/reader.cc b/reader.cc
@@ -321,7 +321,6 @@ MemReader::readULEB128(Off off) const
 std::pair<intmax_t, size_t>
 MemReader::readSLEB128(Off off) const
 {
-    auto p = reinterpret_cast<const char *>(data) + off;
     return readleb128<intmax_t>(ptroff(data, off));
 }