Temporary fix array destroy problem.

Author: jmjoy

.gitignore

@@ -3,4 +3,4 @@
 Cargo.lock
 /.cargo
 /vendor
-/core
+core

phper-alloc/src/lib.rs

@@ -13,6 +13,8 @@ use std::{
     ops::{Deref, DerefMut},
 };
 
+// TODO Add ERc, for refcounted type.
+
 /// The item which can be placed into container [EBox].
 pub trait EAllocatable {
     /// The method to free the heap allocated by `emalloc`, should call `efree` at the end.

phper-macros/src/lib.rs

@@ -8,7 +8,7 @@ The proc-macros for [phper](https://crates.io/crates/phper).
 [Unlicense](https://github.com/jmjoy/phper/blob/master/LICENSE).
 */
 
-// TODO Write a bridge macro for easy usage about register functions and classes.
+// TODO Write a bridge macro for easy usage about register functions and classes, like `cxx`.
 
 mod alloc;
 mod inner;

phper-test/src/lib.rs

@@ -95,6 +95,14 @@ pub fn test_php_scripts_with_condition(
             stdout,
             stderr,
         );
+        #[cfg(target_os = "linux")]
+        if output.status.code().is_none() {
+            use std::os::unix::process::ExitStatusExt;
+            println!(
+                "===== signal ======\nExitStatusExt is None, the signal is: {:?}",
+                output.status.signal()
+            );
+        }
         if !condition(output) {
             panic!("test php file `{}` failed", path);
         }

phper/src/arrays.rs

@@ -112,7 +112,10 @@ impl Array {
 impl EAllocatable for Array {
     fn free(ptr: *mut Self) {
         unsafe {
-            zend_hash_destroy(ptr.cast());
+            if (*ptr).inner.gc.refcount == 0 {
+                zend_hash_destroy(ptr.cast());
+                _efree(ptr.cast());
+            }
         }
     }
 }

phper/src/errors.rs

@@ -9,6 +9,7 @@ use anyhow::anyhow;
 use std::{convert::Infallible, error, ffi::FromBytesWithNulError, io, str::Utf8Error};
 
 /// PHP Throwable, can cause throwing an exception when setting to [crate::values::Val].
+///
 /// TODO Write a derive macro for auto extends the item implements for enum.
 pub trait Throwable: error::Error {
     fn class_entry(&self) -> &StatelessClassEntry;

phper/src/strings.rs

@@ -7,7 +7,6 @@ use crate::{
 use std::{os::raw::c_char, slice::from_raw_parts, str, str::Utf8Error};
 
 /// Wrapper of [crate::sys::zend_string].
-/// TODO Refactor to ZendString(EBox<zend_string>).
 #[repr(transparent)]
 pub struct ZendString {
     inner: zend_string,