cleanup

.github/workflows/pull_request.yml

@@ -5074,13 +5074,13 @@ jobs:
     steps:
       - name: Build Fuzzers (${{ matrix.sanitizer }})
         id: build
-        uses: clickhouse/oss-fuzz/infra/cifuzz/actions/build_fuzzers@no-implicit-int-float-conversion
+        uses: clickhouse/oss-fuzz/infra/cifuzz/actions/build_fuzzers@localized.v1
         with:
           oss-fuzz-project-name: 'clickhouse'
           language: c++
           sanitizer: ${{ matrix.sanitizer }}
       - name: Run Fuzzers (${{ matrix.sanitizer }})
-        uses: google/oss-fuzz/infra/cifuzz/actions/run_fuzzers@master
+        uses: clickhouse/oss-fuzz/infra/cifuzz/actions/run_fuzzers@localized.v1
         with:
           oss-fuzz-project-name: 'clickhouse'
           language: c++

CMakeLists.txt

@@ -119,7 +119,7 @@ endif()
 add_library(global-libs INTERFACE)
 
 # We don't want to instrument everything with fuzzer, but only specific targets (see below),
-# also, since we build our own llvm, we specifically don't wan't to instrument
+# also, since we build our own llvm, we specifically don't want to instrument
 # libFuzzer library itself - it would result in infinite recursion
 #include (cmake/fuzzer.cmake)
 
@@ -529,9 +529,7 @@ set(CONFIG_INCLUDE_PATH ${CMAKE_CURRENT_BINARY_DIR}/includes/configs CACHE INTER
 include_directories(${CONFIG_INCLUDE_PATH})
 
 # Add as many warnings as possible for our own code.
-message(STATUS "BEFORE WARNINGS")
 include (cmake/warnings.cmake)
-message(STATUS "AFTER WARNINGS")
 include (cmake/print_flags.cmake)
 
 if (ENABLE_RUST)

cmake/add_warning.cmake

@@ -3,7 +3,6 @@ include (CheckCCompilerFlag)
 
 # Try to add -Wflag if compiler supports it
 macro (add_warning flag)
-    message(STATUS "ADD WARNING ${flag}")
     string (REPLACE "-" "_" underscored_flag ${flag})
     string (REPLACE "+" "x" underscored_flag ${underscored_flag})
 

cmake/warnings.cmake

@@ -1,5 +1,3 @@
-message(STATUS "INSIDE WARNINGS 0")
-
 # Our principle is to enable as many warnings as possible and always do it with "warnings as errors" flag.
 #
 # But it comes with some cost:
@@ -9,7 +7,6 @@ message(STATUS "INSIDE WARNINGS 0")
 # - sometimes warnings from 3rd party libraries may come from macro substitutions in our code
 #   and we have to wrap them with #pragma GCC/clang diagnostic ignored
 
-message(STATUS "INSIDE WARNINGS")
 set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra")
 
 # Control maximum size of stack frames. It can be important if the code is run in fibers with small stack size.
@@ -21,7 +18,6 @@ endif ()
 if (COMPILER_CLANG)
     # Add some warnings that are not available even with -Wall -Wextra -Wpedantic.
     # We want to get everything out of the compiler for code quality.
-    message(STATUS "WARNINGS SETUP")
     add_warning(everything)
     add_warning(pedantic)
     no_warning(zero-length-array)
@@ -31,7 +27,6 @@ if (COMPILER_CLANG)
     no_warning(sign-conversion)
     no_warning(implicit-int-conversion)
     no_warning(implicit-int-float-conversion)
-    no_warning(implicit-const-int-float-conversion)
     no_warning(ctad-maybe-unsupported) # clang 9+, linux-only
     no_warning(disabled-macro-expansion)
     no_warning(documentation-unknown-command)

src/Compression/fuzzers/lz4_decompress_fuzzer.cpp

@@ -1,4 +1,3 @@
-#include "base/types.h"
 #include <string>
 
 #include <Compression/ICompressionCodec.h>