feat(php): Add PHP 8.5 support

Author: ptondereau

.github/workflows/build.yml

@@ -87,7 +87,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        php: ["8.1", "8.2", "8.3", "8.4"]
+        php: ["8.1", "8.2", "8.3", "8.4", "8.5"]
         rust: [stable, nightly]
         clang: ["15", "17"]
         phpts: [ts, nts]
@@ -173,7 +173,7 @@ jobs:
     runs-on: ubuntu-latest
     env:
       clang: "17"
-      php_version: "8.4"
+      php_version: "8.5"
     steps:
       - name: Checkout code
         uses: actions/checkout@v5
@@ -186,7 +186,7 @@ jobs:
           debug: true
 
       - name: Install libphp-embed
-        run: sudo apt update -y && sudo apt install -y libphp8.4-embed
+        run: sudo apt update -y && sudo apt install -y libphp8.5-embed
 
       - name: Setup Rust
         uses: dtolnay/rust-toolchain@master

Dockerfile

@@ -1,5 +1,5 @@
 FROM rust:latest AS base
-ARG PHP_VERSION=8.4
+ARG PHP_VERSION=8.5
 WORKDIR /tmp
 RUN <<EOF
 set -e

allowed_bindings.rs

@@ -271,6 +271,7 @@ bind! {
     zend_hash_get_current_key_zval_ex,
     zend_hash_get_current_data_ex,
     zend_hash_move_backwards_ex,
+    zend_hash_key_type,
     zend_array_count,
     gc_possible_root,
     ZEND_ACC_NOT_SERIALIZABLE,

build.rs

@@ -268,6 +268,7 @@ enum ApiVersion {
     Php82 = 2022_08_29,
     Php83 = 2023_08_31,
     Php84 = 2024_09_24,
+    Php85 = 2025_09_25,
 }
 
 impl ApiVersion {
@@ -285,7 +286,7 @@ impl ApiVersion {
 
     /// Returns the maximum API version supported by ext-php-rs.
     pub const fn max() -> Self {
-        ApiVersion::Php84
+        ApiVersion::Php85
     }
 
     pub fn versions() -> Vec<Self> {
@@ -295,6 +296,7 @@ impl ApiVersion {
             ApiVersion::Php82,
             ApiVersion::Php83,
             ApiVersion::Php84,
+            ApiVersion::Php85,
         ]
     }
 
@@ -313,6 +315,7 @@ impl ApiVersion {
             ApiVersion::Php82 => "php82",
             ApiVersion::Php83 => "php83",
             ApiVersion::Php84 => "php84",
+            ApiVersion::Php85 => "php85",
         }
     }
 
@@ -323,6 +326,7 @@ impl ApiVersion {
             ApiVersion::Php82 => "EXT_PHP_RS_PHP_82",
             ApiVersion::Php83 => "EXT_PHP_RS_PHP_83",
             ApiVersion::Php84 => "EXT_PHP_RS_PHP_84",
+            ApiVersion::Php85 => "EXT_PHP_RS_PHP_85",
         }
     }
 }
@@ -344,7 +348,10 @@ impl TryFrom<u32> for ApiVersion {
             x if ((ApiVersion::Php83 as u32)..(ApiVersion::Php84 as u32)).contains(&x) => {
                 Ok(ApiVersion::Php83)
             }
-            x if (ApiVersion::Php84 as u32) == x => Ok(ApiVersion::Php84),
+            x if ((ApiVersion::Php84 as u32)..(ApiVersion::Php85 as u32)).contains(&x) => {
+                Ok(ApiVersion::Php84)
+            }
+            x if (ApiVersion::Php85 as u32) == x => Ok(ApiVersion::Php85),
             version => Err(anyhow!(
                 "The current version of PHP is not supported. Current PHP API version: {}, requires a version between {} and {}",
                 version,
@@ -371,7 +378,7 @@ fn check_php_version(info: &PHPInfo) -> Result<()> {
     // The PHP version cfg flags should also stack - if you compile on PHP 8.2 you
     // should get both the `php81` and `php82` flags.
     println!(
-        "cargo::rustc-check-cfg=cfg(php80, php81, php82, php83, php84, php_zts, php_debug, docs)"
+        "cargo::rustc-check-cfg=cfg(php80, php81, php82, php83, php84, php85, php_zts, php_debug, docs)"
     );
 
     if version == ApiVersion::Php80 {
@@ -416,6 +423,7 @@ fn main() -> Result<()> {
         println!("cargo:rustc-cfg=php82");
         println!("cargo:rustc-cfg=php83");
         println!("cargo:rustc-cfg=php84");
+        println!("cargo:rustc-cfg=php85");
         std::fs::copy("docsrs_bindings.rs", out_path)
             .expect("failed to copy docs.rs stub bindings to out directory");
         return Ok(());

docsrs_bindings.rs

@@ -107,13 +107,26 @@ impl IntoConst for &str {
         flags: GlobalConstantFlags,
     ) -> Result<()> {
         unsafe {
-            zend_register_string_constant(
-                CString::new(name)?.as_ptr(),
-                name.len() as _,
-                CString::new(*self)?.as_ptr(),
-                flags.bits().try_into()?,
-                module_number,
-            );
+            #[cfg(php85)]
+            {
+                let _ = zend_register_string_constant(
+                    CString::new(name)?.as_ptr(),
+                    name.len() as _,
+                    CString::new(*self)?.as_ptr(),
+                    flags.bits().try_into()?,
+                    module_number,
+                );
+            }
+            #[cfg(not(php85))]
+            {
+                zend_register_string_constant(
+                    CString::new(name)?.as_ptr(),
+                    name.len() as _,
+                    CString::new(*self)?.as_ptr(),
+                    flags.bits().try_into()?,
+                    module_number,
+                );
+            }
         };
         Ok(())
     }
@@ -127,13 +140,26 @@ impl IntoConst for bool {
         flags: GlobalConstantFlags,
     ) -> Result<()> {
         unsafe {
-            zend_register_bool_constant(
-                CString::new(name)?.as_ptr(),
-                name.len() as _,
-                *self,
-                flags.bits().try_into()?,
-                module_number,
-            );
+            #[cfg(php85)]
+            {
+                let _ = zend_register_bool_constant(
+                    CString::new(name)?.as_ptr(),
+                    name.len() as _,
+                    *self,
+                    flags.bits().try_into()?,
+                    module_number,
+                );
+            }
+            #[cfg(not(php85))]
+            {
+                zend_register_bool_constant(
+                    CString::new(name)?.as_ptr(),
+                    name.len() as _,
+                    *self,
+                    flags.bits().try_into()?,
+                    module_number,
+                );
+            }
         };
         Ok(())
     }
@@ -150,15 +176,29 @@ macro_rules! into_const_num {
                 module_number: i32,
                 flags: GlobalConstantFlags,
             ) -> Result<()> {
-                Ok(unsafe {
-                    $fn(
-                        CString::new(name)?.as_ptr(),
-                        name.len() as _,
-                        (*self).into(),
-                        flags.bits().try_into()?,
-                        module_number,
-                    )
-                })
+                unsafe {
+                    #[cfg(php85)]
+                    {
+                        let _ = $fn(
+                            CString::new(name)?.as_ptr(),
+                            name.len() as _,
+                            (*self).into(),
+                            flags.bits().try_into()?,
+                            module_number,
+                        );
+                    }
+                    #[cfg(not(php85))]
+                    {
+                        $fn(
+                            CString::new(name)?.as_ptr(),
+                            name.len() as _,
+                            (*self).into(),
+                            flags.bits().try_into()?,
+                            module_number,
+                        );
+                    }
+                };
+                Ok(())
             }
         }
     };

src/constant.rs

@@ -15,6 +15,9 @@ use crate::{
     types::Zval,
 };
 
+#[cfg(php85)]
+use crate::ffi::zend_hash_key_type_HASH_KEY_NON_EXISTENT;
+
 /// Immutable iterator upon a reference to a hashtable.
 pub struct Iter<'a> {
     ht: &'a ZendHashTable,
@@ -59,11 +62,16 @@ impl<'a> Iter<'a> {
             zend_hash_get_current_key_type_ex(ptr::from_ref(self.ht).cast_mut(), &raw mut self.pos)
         };
 
-        // Key type `-1` is ???
-        // Key type `1` is string
-        // Key type `2` is long
-        // Key type `3` is null meaning the end of the array
-        if key_type == -1 || key_type == 3 {
+        // Key type `-1` (pre-PHP 8.5) or HASH_KEY_NON_EXISTENT (PHP 8.5+) is ???
+        // Key type `1` is string (HASH_KEY_IS_STRING)
+        // Key type `2` is long (HASH_KEY_IS_LONG)
+        // Key type `3` is null meaning the end of the array (HASH_KEY_NON_EXISTENT)
+        #[cfg(php85)]
+        if key_type == zend_hash_key_type_HASH_KEY_NON_EXISTENT {
+            return None;
+        }
+        #[cfg(not(php85))]
+        if key_type == 3 {
             return None;
         }
 
@@ -157,7 +165,12 @@ impl DoubleEndedIterator for Iter<'_> {
             zend_hash_get_current_key_type_ex(ptr::from_ref(self.ht).cast_mut(), &raw mut self.pos)
         };
 
-        if key_type == -1 {
+        #[cfg(php85)]
+        if key_type == zend_hash_key_type_HASH_KEY_NON_EXISTENT {
+            return None;
+        }
+        #[cfg(not(php85))]
+        if key_type == 3 {
             return None;
         }
 

src/types/array/iterators.rs

@@ -9,5 +9,5 @@ docker buildx build \
   --platform linux/amd64 \
   --target docsrs_bindings \
   -o type=local,dest=. \
-  --build-arg PHP_VERSION=8.4 \
+  --build-arg PHP_VERSION=8.5 \
   .