Commit Graph

1735 Commits

Author SHA1 Message Date
Ferry Huberts
7abf37dc9e Process review comments
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-19 08:20:41 +02:00
Leonid Stryzhevskyi
ae908b82ee
Merge pull request #827 from fhuberts/fix-Wsuggest-attribute=cold
Enable compiler warnings (-Wsuggest-attribute=cold)
2023-08-19 03:27:37 +03:00
Leonid Stryzhevskyi
217515591e
Merge pull request #824 from fhuberts/fix-Wshadow
Fix compiler warnings (-Wshadow)
2023-08-19 03:15:28 +03:00
Leonid Stryzhevskyi
4839095677
Merge pull request #823 from fhuberts/fix-Wdouble-promotion
Fix compiler warnings (-Wdouble-promotion)
2023-08-19 03:14:15 +03:00
Ferry Huberts
f49b3bf0e5 Fix compiler warnings (-Wuseless-cast)
The compiler didn't catch this one in the oatpp build, but did catch it
in the project in which we integrated oatpp.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-17 10:09:52 +02:00
Ferry Huberts
fabcc79ee1 Fix compiler warnings (-Wsign-compare)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-17 08:49:58 +02:00
Ferry Huberts
0bdc3b6aaa Enable compiler warnings (-Wsuggest-attribute=cold)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-17 08:46:51 +02:00
Ferry Huberts
f6f2d27b85 Fix compiler warnings (-Wlogical-op)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-17 08:46:15 +02:00
Ferry Huberts
4652ad8682 Fix compiler warnings (-Wsuggest-attribute=noreturn)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-17 08:45:52 +02:00
Ferry Huberts
91a7b3ef9c Fix compiler warnings (-Wshadow)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-17 08:45:19 +02:00
Ferry Huberts
4b82a5cadd Fix compiler warnings (-Wdouble-promotion)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-17 08:44:35 +02:00
Leonid Stryzhevskyi
78f2232012
Merge pull request #822 from fhuberts/fix-Wdangling-reference
Fix compiler warnings (-Wdangling-reference)
2023-08-17 01:57:59 +03:00
Leonid Stryzhevskyi
75fc72b02c
Merge pull request #821 from fhuberts/fix-Wconversion
Fix compiler warnings (-Wconversion)
2023-08-17 01:56:31 +03:00
Leonid Stryzhevskyi
61f394ad4f
Merge pull request #820 from fhuberts/fix-Warith-conversion
Enable compiler warnings (-Warith-conversion)
2023-08-17 01:54:41 +03:00
Leonid Stryzhevskyi
ddd05b6ef5
Merge pull request #819 from fhuberts/fix-Wsuggest-attribute=format
Fix compiler warnings (-Wsuggest-attribute=format)
2023-08-17 01:53:28 +03:00
Leonid Stryzhevskyi
adb6ac0184
Merge pull request #818 from fhuberts/fix-Wuseless-cast
Fix compiler warnings (-Wuseless-cast)
2023-08-17 01:51:00 +03:00
Ferry Huberts
9f97e4ae61 Fix compiler warnings (-Wdangling-reference)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-16 20:44:41 +02:00
Ferry Huberts
129dfeb782 Forgot to remove the flag -Wno-suggest-attribute=format
No impact on the build.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-16 20:25:24 +02:00
Ferry Huberts
85ee98509a Fix compiler warnings (-Wconversion)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-16 14:02:42 +02:00
Ferry Huberts
7b3c54426c Enable compiler warnings (-Warith-conversion)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-16 14:00:42 +02:00
Ferry Huberts
6dfe302829 Fix compiler warnings (-Wsuggest-attribute=format)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-16 13:56:27 +02:00
Ferry Huberts
f116a9619a Fix compiler warnings (-Wuseless-cast)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-16 13:48:14 +02:00
Leonid Stryzhevskyi
96bff7a7ed
Merge pull request #816 from fhuberts/compiler-flags-are-for-gcc
Compiler flags are for gcc
2023-08-16 02:17:01 +03:00
Leonid Stryzhevskyi
fd82260da6
Merge pull request #815 from fhuberts/fix-2-trivial-clang-warnings
Fix 2 trivial clang warnings
2023-08-16 02:15:21 +03:00
Ferry Huberts
2b4f8f023b These are C++ compiler flags
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-15 15:14:30 +02:00
Ferry Huberts
2a6c8e2692 Only test compiler flags when needed
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-15 13:52:23 +02:00
Ferry Huberts
0d2985cacd Add some comments
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-14 23:10:08 +02:00
Ferry Huberts
6553029e2f Only apply the compiler flags for GNU
That's what they are originally tuned for.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-14 23:10:08 +02:00
Ferry Huberts
16392c700b Convert all 'add_compiler_flags' to 'add_cxx_compiler_flags'
This codebase only contains C++ code.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-14 23:10:08 +02:00
Ferry Huberts
c0f0b46ed8 Remove a superfluous line
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-14 23:10:08 +02:00
Ferry Huberts
0b96ccdd5b Fix Clang compiler warnings (-Wzero-as-null-pointer-constant)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-14 22:32:40 +02:00
Ferry Huberts
83138fe94d Fix Clang compiler warnings (-Wc++98-compat-extra-semi)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-14 22:32:37 +02:00
Leonid Stryzhevskyi
52b0d0b613
Merge pull request #805 from fhuberts/fix-Wold-style-cast
Fix compiler warnings (-Wold-style-cast)
2023-08-12 04:00:45 +03:00
Ferry Huberts
e872a75594 Process review comments
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-11 10:32:59 +02:00
Leonid Stryzhevskyi
0842dd4133
Merge pull request #809 from fhuberts/fix-Wctor-dtor-privacy
Fix compiler warnings (-Wctor-dtor-privacy)
2023-08-09 02:01:56 +03:00
Ferry Huberts
152edd30d7 Process review comments
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-08 09:01:56 +02:00
Ferry Huberts
967a6906a4 Fix compiler warnings (-Wctor-dtor-privacy)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-07 18:03:01 +02:00
Leonid Stryzhevskyi
ec227380e4
Merge pull request #808 from fhuberts/fix-Wmissing-field-initializers
Fix compiler warnings (-Wmissing-field-initializers)
2023-08-07 17:36:42 +03:00
Leonid Stryzhevskyi
0c1528fdb9
Merge pull request #807 from fhuberts/fix-Wunused-variable
Fix compiler warnings (-Wunused-variable)
2023-08-07 17:33:37 +03:00
Ferry Huberts
c81189a775 Fix compiler warnings (-Wmissing-field-initializers)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-07 11:15:39 +02:00
Ferry Huberts
40f5b11194 Fix compiler warnings (-Wunused-variable)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-07 11:14:57 +02:00
Ferry Huberts
8ff0b0f948 Fix compiler warnings (-Wold-style-cast)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-07 11:14:19 +02:00
Leonid Stryzhevskyi
fd93069799
Merge pull request #802 from fhuberts/add-compiler-flags
Explicitly add compiler flags
2023-08-07 01:14:01 +03:00
Ferry Huberts
1ab56bc771 Try to fix the Windows build
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-06 10:03:28 +02:00
Ferry Huberts
a52d1500fe More hardening compiler flags
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-05 15:58:11 +02:00
Ferry Huberts
579e9f3319 Add some more possible compiler flags
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-05 15:58:05 +02:00
Ferry Huberts
c50959f6e6 Move compiler flags into a separate cmake file
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-05 11:43:20 +02:00
Leonid Stryzhevskyi
38e02370b9
Merge pull request #803 from fhuberts/fix-Winfinite-recursion
Fix compiler warnings (-Winfinite-recursion)
2023-08-05 00:28:10 +03:00
Leonid Stryzhevskyi
73bc335121
Merge pull request #801 from fhuberts/fix-Wcatch-value
Fix compiler warnings (-Wcatch-value=)
2023-08-05 00:20:58 +03:00
Ferry Huberts
fdbf3b83eb Fix compiler warnings (-Winfinite-recursion)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
2023-08-04 15:57:58 +02:00