Remove/Improve some TODOs (#24)

Author: havogt

CMakeLists.txt

@@ -1,4 +1,5 @@
 cmake_minimum_required(VERSION 3.12.4)
+# when increasing the minium required version, consider updating in examples as well
 
 file(STRINGS "version.txt" __CPP_BINDGEN_VERSION)
 project(cpp_bindgen VERSION ${__CPP_BINDGEN_VERSION} LANGUAGES CXX)

example/simple/CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.12.4) #TODO replace with generated version
+cmake_minimum_required(VERSION 3.12.4)
 project(simple_example LANGUAGES CXX C Fortran)
 
 # 1) find installed c_bindings version
@@ -10,4 +10,3 @@ cpp_bindgen_add_library(simple SOURCES simple.cpp)
 # 3) link the library to a fortran executable
 add_executable(driver driver.f90)
 target_link_libraries(driver simple_fortran)
-

example/simple_fetch_content/CMakeLists.txt

@@ -1,8 +1,7 @@
-cmake_minimum_required(VERSION 3.12.4) #TODO replace with generated version
+cmake_minimum_required(VERSION 3.12.4)
 project(simple_example_with_fetch_content LANGUAGES CXX Fortran)
 
 # 1) fetch cpp_bindgen from the repository
-# TODO: use FetchContent_MakeAvailable() once we require CMake 3.14+
 include(FetchContent)
 FetchContent_Declare(
   cpp_bindgen
@@ -12,7 +11,7 @@ FetchContent_Declare(
   # GIT_REPOSITORY https://github.com/GridTools/cpp_bindgen.git
   # GIT_TAG        master # consider replacing master by a tagged version
 )
-# FetchContent_MakeAvailable(cpp_bindgen) for CMake >= 3.14
+# FetchContent_MakeAvailable(cpp_bindgen) for CMake >= 3.14, otherwise:
 FetchContent_GetProperties(cpp_bindgen)
 if(NOT c_bindings_POPULATED)
   FetchContent_Populate(cpp_bindgen)
@@ -25,4 +24,3 @@ cpp_bindgen_add_library(simple SOURCES simple.cpp)
 # 3) link the generated library to a fortran executable
 add_executable(driver driver.f90)
 target_link_libraries(driver simple_fortran)
-