cmake: remove arguments in endif()

The arguments to `else()` and `endif()` were required before
version 2.6.0. As of CMake 2.6.0 the `else()` and `endif()`
constructs can be empty. The same is true for closing constructs
on `endmacro()`, `endfunction()`, and `endforeach()`. If you
require 2.4.x compatibility, CMake 2.4.3 or greater recognizes the
`CMAKE_ALLOW_LOOSE_LOOP_CONSTRUCTS` option (which is superfluous
in 2.6.0). See [1] and [2].

The patch removes arguments in `endif()`.

1. https://cmake.org/cmake/help/v3.6/command/endif.html
2. https://stackoverflow.com/questions/29959126/why-does-cmake-syntax-have-redundant-parentheses-everywhere/29967305#29967305

Author: ligurio

CMakeLists.txt

@@ -34,7 +34,7 @@ include(SetHardwareArch)
 
 if (ENABLE_LUAJIT_RANDOM_RA AND NOT USE_LUAJIT)
   message(FATAL_ERROR "Option ENABLE_LUAJIT_RANDOM_RA is LuaJIT-specific.")
-endif (ENABLE_LUAJIT_RANDOM_RA AND NOT USE_LUAJIT)
+endif()
 
 if (USE_LUA AND NOT LUA_VERSION)
   set(LUA_VERSION "master")
@@ -70,7 +70,7 @@ endif()
 find_package(Protobuf)
 if (NOT Protobuf_FOUND)
     set(ENABLE_BUILD_PROTOBUF ON)
-endif (NOT Protobuf_FOUND)
+endif()
 
 SetBuildParallelLevel(CMAKE_BUILD_PARALLEL_LEVEL)
 

cmake/CodeCoverage.cmake

@@ -42,7 +42,7 @@ if(USE_LUAJIT)
   # Exclude buildvm source code, it's a project's build infrastructure.
   set(GCOVR_OPTIONS ${GCOVR_OPTIONS} --exclude ".*/host/")
   set(GCOVR_OPTIONS ${GCOVR_OPTIONS} --object-directory ${LUA_SOURCE_DIR}/src)
-endif (USE_LUAJIT)
+endif ()
 
 file(MAKE_DIRECTORY ${COVERAGE_DIR})
 add_custom_target(${target_name})
@@ -61,7 +61,7 @@ if(USE_LUAJIT)
   # CMake cannot remove recursively by globbing.
   # Files 'src/host/*.gcda' are not used for building coverage report.
   set(GCDA_FILES "${LUA_SOURCE_DIR}/src/*.gcda")
-endif(USE_LUAJIT)
+endif()
 add_custom_target(coverage-reset
   COMMENT "Reset code coverage counters"
   COMMAND ${CMAKE_COMMAND} -E rm -f ${GCDA_FILES}

libluamut/CMakeLists.txt

@@ -5,7 +5,7 @@ if (ENABLE_COV)
              -fcoverage-mapping -ftest-coverage)
   set(LDFLAGS ${LDFLAGS} -fprofile-instr-generate -fprofile-arcs
               -fcoverage-mapping -ftest-coverage)
-endif (ENABLE_COV)
+endif()
 
 set(LIB_LUA_MUTATE lua_mutate)
 add_library(${LIB_LUA_MUTATE} STATIC mutate.c)

tests/CMakeLists.txt

@@ -1,4 +1,4 @@
 add_subdirectory(capi)
 if (ENABLE_BONUS_TESTS)
   add_subdirectory(lapi)
-endif (ENABLE_BONUS_TESTS)
+endif()

tests/capi/CMakeLists.txt

@@ -29,19 +29,19 @@ target_link_libraries(
 
 if (CMAKE_BUILD_TYPE STREQUAL "Debug")
   set(LDFLAGS "${LDFLAGS} ${CMAKE_C_FLAGS_DEBUG}")
-endif (CMAKE_BUILD_TYPE STREQUAL "Debug")
+endif()
 
 if (ENABLE_ASAN)
   set(LDFLAGS "${LDFLAGS} -fsanitize=address")
-endif (ENABLE_ASAN)
+endif()
 
 if (ENABLE_UBSAN)
   set(LDFLAGS "${LDFLAGS} -fsanitize=undefined")
-endif (ENABLE_UBSAN)
+endif()
 
 if (ENABLE_COV)
   set(LDFLAGS "${LDFLAGS} -fprofile-instr-generate -fcoverage-mapping")
-endif (ENABLE_COV)
+endif()
 
 set(DEFAULT_RUNS_NUMBER 5)
 
@@ -96,14 +96,14 @@ function(create_test)
     set_tests_properties(${test_name} PROPERTIES
       ENVIRONMENT "ASAN_OPTIONS='detect_invalid_pointer_pairs=2'"
     )
-  endif (USE_LUA)
+  endif()
   set_tests_properties(${test_name} PROPERTIES
     LABELS capi
   )
 
   if (USE_LUAJIT)
     target_compile_definitions(${test_name} PUBLIC LUAJIT)
-  endif (USE_LUAJIT)
+  endif()
 endfunction()
 
 # These Lua C functions are unsupported by LuaJIT.