Add more php version.

Author: jmjoy

.github/workflows/ci.yml

@@ -21,6 +21,8 @@ jobs:
         os:
           - ubuntu-latest
         php-version:
+          - 7.1
+          - 7.2
           - 7.3
           - 7.4
 
@@ -31,6 +33,9 @@ jobs:
         with:
           php-version: ${{ matrix.php-version }}
 
+      - name: PHP Config
+        run: php-config --version && php-config --phpapi
+
       - name: Checkout
         uses: actions/checkout@v2
 

phper/src/zend/api.rs

@@ -34,15 +34,16 @@ impl<T: 'static> ModuleGlobals<T> {
         on_modify: Option<Mh>,
         modifiable: u32,
     ) -> zend_ini_entry_def {
-        #[cfg(phper_php_version = "7.3")]
-        let modifiable = modifiable as std::os::raw::c_uchar;
-        #[cfg(phper_php_version = "7.2")]
-        let modifiable = modifiable as std::os::raw::c_int;
-
-        #[cfg(phper_php_version = "7.3")]
-        let name_length = name.len() as u16;
-        #[cfg(phper_php_version = "7.2")]
-        let name_length = name.len() as u32;
+        #[cfg(any(
+            phper_php_version = "7.4",
+            phper_php_version = "7.3",
+        ))]
+        let (modifiable, name_length) = (modifiable as std::os::raw::c_uchar, name.len() as u16);
+        #[cfg(any(
+            phper_php_version = "7.2",
+            phper_php_version = "7.1",
+        ))]
+        let (modifiable, name_length) = (modifiable as std::os::raw::c_int, name.len() as u32);
 
         zend_ini_entry_def {
             name: name.as_ptr().cast(),