Fix document tests.

Author: jmjoy

examples/hello/tests/integration.rs

@@ -8,10 +8,7 @@
 // NON-INFRINGEMENT, MERCHANTABILITY OR FIT FOR A PARTICULAR PURPOSE.
 // See the Mulan PSL v2 for more details.
 
-use phper_test::{
-    cli::{test_php_scripts, test_php_scripts_with_lib},
-    utils::get_lib_path_by_example,
-};
+use phper_test::{cli::test_php_scripts_with_lib, utils::get_lib_path_by_example};
 use std::{env, path::Path};
 
 #[test]

examples/http-client/tests/integration.rs

@@ -8,10 +8,7 @@
 // NON-INFRINGEMENT, MERCHANTABILITY OR FIT FOR A PARTICULAR PURPOSE.
 // See the Mulan PSL v2 for more details.
 
-use phper_test::{
-    cli::{test_php_scripts, test_php_scripts_with_lib},
-    utils::get_lib_path_by_example,
-};
+use phper_test::{cli::test_php_scripts_with_lib, utils::get_lib_path_by_example};
 use std::{env, path::Path};
 
 #[test]

examples/http-server/tests/integration.rs

@@ -10,10 +10,7 @@
 
 use hyper::header::CONTENT_TYPE;
 use phper_test::{
-    cli::{
-        test_long_term_php_script_with_condition, test_long_term_php_script_with_condition_and_lib,
-    },
-    utils::get_lib_path_by_example,
+    cli::test_long_term_php_script_with_condition_and_lib, utils::get_lib_path_by_example,
 };
 use reqwest::Client;
 use std::{env, path::Path, thread::sleep, time::Duration};

examples/logging/tests/integration.rs

@@ -8,10 +8,7 @@
 // NON-INFRINGEMENT, MERCHANTABILITY OR FIT FOR A PARTICULAR PURPOSE.
 // See the Mulan PSL v2 for more details.
 
-use phper_test::{
-    cli::{test_php_scripts_with_condition, test_php_scripts_with_condition_and_lib},
-    utils::get_lib_path_by_example,
-};
+use phper_test::{cli::test_php_scripts_with_condition_and_lib, utils::get_lib_path_by_example};
 use std::{env, path::Path, str};
 
 #[test]

phper/src/functions.rs

@@ -13,7 +13,6 @@
 //! TODO Add lambda.
 
 use crate::{
-    alloc::EBox,
     cg,
     classes::Visibility,
     errors::{ArgumentCountError, CallFunctionError, CallMethodError},
@@ -245,7 +244,7 @@ impl ZendFunction {
 
     pub(crate) fn call(
         &mut self, mut object: Option<&mut ZObj>, mut arguments: impl AsMut<[ZVal]>,
-    ) -> crate::Result<EBox<ZVal>> {
+    ) -> crate::Result<ZVal> {
         let arguments = arguments.as_mut();
         let function_handler = self.as_mut_ptr();
 
@@ -422,14 +421,14 @@ pub(crate) const fn create_zend_arg_info(
 ///     Ok(())
 /// }
 /// ```
-pub fn call(fn_name: &str, arguments: impl AsMut<[ZVal]>) -> crate::Result<EBox<ZVal>> {
+pub fn call(fn_name: &str, arguments: impl AsMut<[ZVal]>) -> crate::Result<ZVal> {
     let mut func = fn_name.into();
     call_internal(&mut func, None, arguments)
 }
 
 pub(crate) fn call_internal(
     func: &mut ZVal, mut object: Option<&mut ZObj>, mut arguments: impl AsMut<[ZVal]>,
-) -> crate::Result<EBox<ZVal>> {
+) -> crate::Result<ZVal> {
     let func_ptr = func.as_mut_ptr();
     let arguments = arguments.as_mut();
 
@@ -469,8 +468,8 @@ pub(crate) fn call_internal(
 pub(crate) fn call_raw_common(
     call_fn: impl FnOnce(&mut ZVal) -> bool, name_fn: impl FnOnce() -> crate::Result<String>,
     object: Option<&mut ZObj>,
-) -> crate::Result<EBox<ZVal>> {
-    let mut ret = EBox::new(ZVal::from(()));
+) -> crate::Result<ZVal> {
+    let mut ret = ZVal::default();
 
     if call_fn(&mut ret) && !ret.get_type_info().is_undef() {
         return Ok(ret);

phper/src/objects.rs

@@ -143,16 +143,19 @@ impl ZObj {
     /// ```
     /// use phper::{alloc::EBox, classes::ClassEntry, values::ZVal};
     ///
-    /// fn example() -> phper::Result<EBox<Val>> {
-    ///     let mut memcached = StatelessClassEntry::from_globals("Memcached")?.new_object(&mut [])?;
-    ///     memcached.call("addServer", &mut [Val::new("127.0.0.1"), Val::new(11211)])?;
-    ///     let r = memcached.call("get", &mut [Val::new("hello")])?;
+    /// fn example() -> phper::Result<ZVal> {
+    ///     let mut memcached = ClassEntry::from_globals("Memcached")?.new_object(&mut [])?;
+    ///     memcached.call(
+    ///         "addServer",
+    ///         &mut [ZVal::from("127.0.0.1"), ZVal::from(11211)],
+    ///     )?;
+    ///     let r = memcached.call("get", &mut [ZVal::from("hello")])?;
     ///     Ok(r)
     /// }
     /// ```
     pub fn call(
         &mut self, method_name: &str, arguments: impl AsMut<[ZVal]>,
-    ) -> crate::Result<EBox<ZVal>> {
+    ) -> crate::Result<ZVal> {
         let mut method = method_name.into();
 
         unsafe {

phper/src/values.rs

@@ -327,7 +327,7 @@ impl ZVal {
     /// # Errors
     ///
     /// Return Err when self is not callable.
-    pub fn call(&mut self, arguments: impl AsMut<[ZVal]>) -> crate::Result<EBox<ZVal>> {
+    pub fn call(&mut self, arguments: impl AsMut<[ZVal]>) -> crate::Result<ZVal> {
         call_internal(self, None, arguments)
     }
 }