From 43ad10b04d7c1384ebf63fb718e60d253de172b4 Mon Sep 17 00:00:00 2001 From: Dmitri Petrov Date: Thu, 4 Jan 2024 17:37:59 -0800 Subject: [PATCH] CMake Fixes (#312) --- CMakeLists.txt | 4 +--- db/compaction/compaction_job_test.cc | 2 +- third-party/gtest-1.8.1/fused-src/gtest/gtest-all.cc | 2 +- 3 files changed, 3 insertions(+), 5 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 228c7677e4a..96708b05f11 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -197,7 +197,7 @@ if(MSVC) set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /Zi /nologo /EHsc /GS /Gd /GR /GF /fp:precise /Zc:wchar_t /Zc:forScope /errorReport:queue") set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /FC /d2Zi+ /W4 /wd4127 /wd4800 /wd4996 /wd4351 /wd4100 /wd4204 /wd4324") else() - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -W -Wextra -Wall -pthread") + set(CMAKE_CXX_FLAGS "-W -Wextra -Wall ${CMAKE_CXX_FLAGS} -pthread") set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wsign-compare -Wshadow -Wno-unused-parameter -Wno-unused-variable -Woverloaded-virtual -Wnon-virtual-dtor -Wno-missing-field-initializers -Wno-strict-aliasing -Wno-invalid-offsetof") if(CMAKE_SYSTEM_PROCESSOR MATCHES "x86_64") set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wstrict-prototypes") @@ -989,7 +989,6 @@ set(SOURCES cloud/cloud_scheduler.cc cloud/cloud_storage_provider.cc cloud/cloud_file_deletion_scheduler.cc - db/db_impl/db_impl_remote_compaction.cc db/db_impl/replication_codec.cc) list(APPEND SOURCES @@ -1327,7 +1326,6 @@ if(WITH_TESTS) cloud/db_cloud_test.cc cloud/cloud_manifest_test.cc cloud/cloud_scheduler_test.cc - cloud/remote_compaction_test.cc cloud/replication_test.cc db/blob/blob_counting_iterator_test.cc db/blob/blob_file_addition_test.cc diff --git a/db/compaction/compaction_job_test.cc b/db/compaction/compaction_job_test.cc index 0f0c5daf776..008b3d2d1fd 100644 --- a/db/compaction/compaction_job_test.cc +++ b/db/compaction/compaction_job_test.cc @@ -374,7 +374,7 @@ class CompactionJobTestBase : public testing::Test { } else if (table_type_ == TableTypeForTest::kMockTable) { file_size = 10; EXPECT_OK(mock_table_factory_->CreateMockTable( - env_, GenerateFileName(file_number), std::move(contents))); + env_, GenerateFileName(file_number), contents)); } else { assert(false); } diff --git a/third-party/gtest-1.8.1/fused-src/gtest/gtest-all.cc b/third-party/gtest-1.8.1/fused-src/gtest/gtest-all.cc index 9f2b3d56530..98615ea3b9f 100644 --- a/third-party/gtest-1.8.1/fused-src/gtest/gtest-all.cc +++ b/third-party/gtest-1.8.1/fused-src/gtest/gtest-all.cc @@ -8676,7 +8676,7 @@ static void StackLowerThanAddress(const void* ptr, bool* result) { // Make sure AddressSanitizer does not tamper with the stack here. GTEST_ATTRIBUTE_NO_SANITIZE_ADDRESS_ static bool StackGrowsDown() { - int dummy; + int dummy = 1; bool result; StackLowerThanAddress(&dummy, &result); return result;