🛠 Continue to fight off terrible-by-default GNU Warnings

soasis · Dec 24, 2022 · 9af1129 · 9af1129
1 parent 07d1109
commit 9af1129
Show file tree

Hide file tree

Showing 17 changed files with 19 additions and 16 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -93,6 +93,8 @@ if(ZTD_TEXT_IS_TOP_LEVEL_PROJECT)
 		set(CMAKE_C_STANDARD 11)
 	endif()
 
+	set(CMAKE_OBJECT_PATH_MAX 1024)
+
 	if(ZTD_TEXT_BENCHMARKS OR ZTD_TEXT_EXAMPLES OR ZTD_TEXT_TESTS OR ZTD_TEXT_SCRATCH)
 		# normal flags
 		check_compiler_flag(disable-permissive MSVC /permissive- GCC -pedantic)
@@ -117,6 +119,7 @@ if(ZTD_TEXT_IS_TOP_LEVEL_PROJECT)
 		#   https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95353
 		if (CMAKE_CXX_COMPILER_ID MATCHES "GNU" AND NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 7.0)
 			check_compiler_diagnostic(stringop-overflow)
+			check_compiler_diagnostic(stringop-overread)
 			check_compiler_diagnostic(array-bounds)
 		endif()
 	endif()

diff --git a/benchmarks/barrier/CMakeLists.txt b/benchmarks/barrier/CMakeLists.txt
@@ -58,7 +58,7 @@ target_compile_options(ztd.text.benchmarks.barrier
 	${--warn-extra}
 	${--warn-errors}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_link_libraries(ztd.text.benchmarks.barrier

diff --git a/benchmarks/conversion_speed/CMakeLists.txt b/benchmarks/conversion_speed/CMakeLists.txt
@@ -65,7 +65,7 @@ function (generate_converion_speed_benchmark_targets name data_name title)
 			${--warn-extra}
 			${--warn-errors}
 			${--allow-alignas-extra-padding}
-			${--allow-stringop-overflow}
+			${--allow-stringop-overflow} ${--allow-stringop-overread}
 			${--allow-array-bounds}
 			)
 	target_compile_definitions(ztd.text.benchmarks.conversion_speed.${name}

diff --git a/benchmarks/function_form/CMakeLists.txt b/benchmarks/function_form/CMakeLists.txt
@@ -45,7 +45,7 @@ function (generate_function_form_benchmark_targets name data_name title)
 		${--warn-all}
 		${--warn-extra}
 		${--allow-alignas-extra-padding}
-		${--allow-stringop-overflow}
+		${--allow-stringop-overflow} ${--allow-stringop-overread}
 		${--allow-array-bounds}
 		# MSVC is BROKEN
 		# ${--warn-errors}

diff --git a/examples/basic/CMakeLists.txt b/examples/basic/CMakeLists.txt
@@ -73,7 +73,7 @@ foreach (example_source_name ${ztd.text.examples.basic.sources})
 		${--warn-extra}
 		${--warn-errors}
 		${--allow-alignas-extra-padding}
-		${--allow-stringop-overflow}
+		${--allow-stringop-overflow} ${--allow-stringop-overread}
 		${--allow-array-bounds}
 		${--allow-bit-int-extension}
 	)

diff --git a/examples/boost.text/CMakeLists.txt b/examples/boost.text/CMakeLists.txt
@@ -44,7 +44,7 @@ target_compile_options(ztd.text.examples.boost_text
 		${--warn-extra}
 		${--warn-errors}
 		${--allow-alignas-extra-padding}
-		${--allow-stringop-overflow}
+		${--allow-stringop-overflow} ${--allow-stringop-overread}
 		${--allow-array-bounds}
 )
 

diff --git a/examples/documentation/compile_fails/CMakeLists.txt b/examples/documentation/compile_fails/CMakeLists.txt
@@ -46,7 +46,7 @@ foreach (example_name ${ztd.text.examples.documentation.compile_fails.sources})
 		${--warn-extra}
 		${--warn-errors}
 		${--allow-alignas-extra-padding}
-		${--allow-stringop-overflow}
+		${--allow-stringop-overflow} ${--allow-stringop-overread}
 		${--allow-array-bounds}
 	)
 	set_target_properties(${example_target}

diff --git a/examples/documentation/quick/basic/CMakeLists.txt b/examples/documentation/quick/basic/CMakeLists.txt
@@ -49,7 +49,7 @@ foreach (example_source_name ${ztd.text.examples.documentation.quick.basic.sourc
 		${--warn-extra}
 		${--warn-errors}
 		${--allow-alignas-extra-padding}
-		${--allow-stringop-overflow}
+		${--allow-stringop-overflow} ${--allow-stringop-overread}
 		${--allow-array-bounds}
 	)
 

diff --git a/examples/documentation/snippets/CMakeLists.txt b/examples/documentation/snippets/CMakeLists.txt
@@ -45,7 +45,7 @@ foreach (example_source_name ${ztd.text.examples.documentation.snippets.sources}
 		${--warn-all}
 		${--warn-errors}
 		${--allow-alignas-extra-padding}
-		${--allow-stringop-overflow}
+		${--allow-stringop-overflow} ${--allow-stringop-overread}
 		${--allow-array-bounds}
 	)
 	target_include_directories(${example_target}

diff --git a/examples/shift_jis/CMakeLists.txt b/examples/shift_jis/CMakeLists.txt
@@ -63,7 +63,7 @@ target_compile_options(ztd.text.examples.shift_jis
 	${--warn-extra}
 	${--warn-errors}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_include_directories(ztd.text.examples.shift_jis

diff --git a/tests/additional_encodings/CMakeLists.txt b/tests/additional_encodings/CMakeLists.txt
@@ -70,7 +70,7 @@ target_compile_options(ztd.text.tests.additional_encodings
 	${--warn-extra}
 	${--warn-errors}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_include_directories(ztd.text.tests.additional_encodings

diff --git a/tests/basic_compile_time/CMakeLists.txt b/tests/basic_compile_time/CMakeLists.txt
@@ -52,7 +52,7 @@ target_compile_options(ztd.text.tests.basic_compile_time
 	${--extra-constexpr-depth}
 	${--extra-constexpr-steps}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_include_directories(ztd.text.tests.basic_compile_time

diff --git a/tests/basic_run_time/CMakeLists.txt b/tests/basic_run_time/CMakeLists.txt
@@ -48,7 +48,7 @@ target_compile_options(ztd.text.tests.basic_run_time
 	${--warn-extra}
 	${--warn-errors}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_include_directories(ztd.text.tests.basic_run_time

diff --git a/tests/compile_fails/CMakeLists.txt b/tests/compile_fails/CMakeLists.txt
@@ -47,7 +47,7 @@ foreach (test_name ${ztd.text.tests.tests.compile_fails.sources})
 		${--warn-extra}
 		${--warn-errors}
 		${--allow-alignas-extra-padding}
-		${--allow-stringop-overflow}
+		${--allow-stringop-overflow} ${--allow-stringop-overread}
 		${--allow-array-bounds}
 	)
 	target_link_libraries(${test_target}

diff --git a/tests/iconv/CMakeLists.txt b/tests/iconv/CMakeLists.txt
@@ -52,7 +52,7 @@ target_compile_options(ztd.text.tests.iconv
 	${--warn-extra}
 	${--warn-errors}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_include_directories(ztd.text.tests.iconv

diff --git a/tests/inclusion/CMakeLists.txt b/tests/inclusion/CMakeLists.txt
@@ -45,7 +45,7 @@ target_compile_options(ztd.text.tests.inclusion
 	${--warn-extra}
 	${--warn-errors}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_link_libraries(ztd.text.tests.inclusion

diff --git a/tests/tiny_buffer/CMakeLists.txt b/tests/tiny_buffer/CMakeLists.txt
@@ -51,7 +51,7 @@ target_compile_options(ztd.text.tests.tiny_buffer
 	${--warn-extra}
 	${--warn-errors}
 	${--allow-alignas-extra-padding}
-	${--allow-stringop-overflow}
+	${--allow-stringop-overflow} ${--allow-stringop-overread}
 	${--allow-array-bounds}
 )
 target_include_directories(ztd.text.tests.tiny_buffer