Add various improvements

- Updated module docs
- get_cmake_property usages replaced with more common get_property() as
  the former is more for CMake core properties than custom global ones.

Author: petk

cmake/cmake/Flags.cmake

@@ -8,7 +8,7 @@ include(CheckSourceRuns)
 include(CMakePushCheckState)
 include(PHP/CheckCompilerFlag)
 
-get_cmake_property(enabledLanguages ENABLED_LANGUAGES)
+get_property(enabledLanguages GLOBAL PROPERTY ENABLED_LANGUAGES)
 
 # Check for broken GCC optimize-strlen.
 include(PHP/CheckBrokenGccStrlenOpt)

cmake/cmake/Testing.cmake

@@ -16,7 +16,7 @@ block()
     set(parallel -j${processors})
   endif()
 
-  get_cmake_property(extensions PHP_EXTENSIONS)
+  get_property(extensions GLOBAL PROPERTY PHP_EXTENSIONS)
   foreach(extension ${extensions})
     get_target_property(type php_${extension} TYPE)
     if(type MATCHES "^(MODULE|SHARED)_LIBRARY$")

cmake/cmake/modules/PHP/AddCustomCommand.cmake

@@ -32,12 +32,16 @@ php_add_custom_command(
 )
 ```
 
+## Basic usage
+
 It acts similar to `add_custom_command()` and `add_custom_target()`, except that
 when PHP is not found on the system, the DEPENDS argument doesn't add
 dependencies among targets but instead checks their timestamps manually and
 executes the PHP_COMMAND only when needed.
 
 ```cmake
+# CMakeLists.txt
+include(PHP/AddCustomCommand)
 php_add_custom_command(
   php_generate_something
   OUTPUT

cmake/cmake/modules/PHP/CheckAVX512.cmake

@@ -10,8 +10,11 @@ TODO: Adjust checks for MSVC.
 ## Cache variables
 
 * `PHP_HAVE_AVX512_SUPPORTS`
+
   Whether compiler supports AVX-512.
+
 * `PHP_HAVE_AVX512_VBMI_SUPPORTS`
+
   Whether compiler supports AVX-512 VBMI.
 #]=============================================================================]
 

cmake/cmake/modules/PHP/CheckBrokenGccStrlenOpt.cmake

@@ -9,6 +9,7 @@ See: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86914
 ## Cache variables
 
 * `PHP_HAVE_BROKEN_OPTIMIZE_STRLEN`
+
   Whether GCC has broken strlen() optimization.
 #]=============================================================================]
 

cmake/cmake/modules/PHP/CheckBuiltin.cmake

@@ -12,11 +12,11 @@ php_check_builtin(<builtin> <result_var>)
 If builtin `<builtin>` is supported by the C compiler, store the check result in
 the cache variable `<result_var>`.
 
-For example:
+## Basic usage
 
 ```cmake
+# CMakeLists.txt
 include(PHP/CheckBuiltin)
-
 php_check_builtin(__builtin_clz PHP_HAVE_BUILTIN_CLZ)
 ```
 #]=============================================================================]

cmake/cmake/modules/PHP/CheckCopyFileRange.cmake

@@ -8,6 +8,7 @@ only on Linux.
 ## Cache variables
 
 * `HAVE_COPY_FILE_RANGE`
+
   Whether `copy_file_range()` is supported.
 #]=============================================================================]
 

cmake/cmake/modules/PHP/CheckFlushIo.cmake

@@ -6,6 +6,7 @@ Check if flush should be called explicitly after buffered io.
 ## Cache variables
 
 * `HAVE_FLUSHIO`
+
   Whether flush should be called explicitly after a buffered io.
 #]=============================================================================]
 

cmake/cmake/modules/PHP/CheckFopencookie.cmake

@@ -13,8 +13,11 @@ to use `off_t` only, check is left here when using glibc.
 ## Cache variables
 
 * `HAVE_FOPENCOOKIE`
+
   Whether `fopencookie()` and `cookie_io_functions_t` are available.
+
 * `COOKIE_SEEKER_USES_OFF64_T`
+
   Whether `fopencookie` seeker uses the `off64_t` type.
 #]=============================================================================]
 

cmake/cmake/modules/PHP/CheckGetaddrinfo.cmake

@@ -6,11 +6,13 @@ Check for working `getaddrinfo()`.
 ## Cache variables
 
 * `HAVE_GETADDRINFO`
+
   Whether `getaddrinfo()` function is working as expected.
 
 IMPORTED target:
 
 * `PHP::CheckGetaddrinfoLibrary`
+
   If there is additional library to be linked for using `getaddrinfo()`.
 #]=============================================================================]
 
@@ -45,8 +47,8 @@ if(libraryForGetaddrinfo)
   )
 endif()
 
-# If the variable HAVE_GETADDRINFO has been overridden (for example, on Windows)
-# or cached in consecutive runs, the module stops here.
+# If the variable HAVE_GETADDRINFO has been overridden the module stops here.
+# For example, on Windows.
 if(HAVE_GETADDRINFO)
   message(CHECK_PASS "yes (cached)")
   return()