Merge pull request cpp-best-practices#2 from Lectem/master

Use target_compile_options and interface library

Author: lefticus

CMakeLists.txt

@@ -1,5 +1,8 @@
 cmake_minimum_required(VERSION 3.2)
 
+# Link this 'library' to use the following warnings
+add_library(project_warnings INTERFACE)
+
 if (CMAKE_COMPILER_IS_GNUCC)
  option(ENABLE_COVERAGE "Enable coverage reporting for gcc/clang" FALSE)
 
@@ -9,18 +12,22 @@ if (CMAKE_COMPILER_IS_GNUCC)
 endif()
 
 if (MSVC)
- add_compile_options(/W4)
+ target_compile_options(project_warnings INTERFACE /W4)
 else()
- add_compile_options(-Wall -Wextra -Wpedantic)
+ target_compile_options(project_warnings INTERFACE -Wall -Wextra -Wpedantic)
 endif()
 
 add_executable(intro main.cpp)
 target_compile_features(intro PRIVATE cxx_lambda_init_captures)
+target_link_libraries(intro PRIVATE project_warnings)
+
 target_link_libraries(intro --coverage)
 
 enable_testing()
 
 add_executable(tester tester.cpp)
+target_link_libraries(tester PRIVATE project_warnings)
+
 target_link_libraries(tester --coverage)
 add_test(Tester tester)