Remove comments and unnecessary code

Author: yoramdelangen

tests/src/integration/variadic_args.php

@@ -20,11 +20,6 @@
 
 // Must have arguments.. so catch ArgumentCountError errors!
 assert_exception_thrown('test_variadic_args');
-// try {
-//     $args = test_variadic_args();
-// } catch (ArgumentCountError $e) {
-//     var_dump($e->getMessage());
-// }
 
 // Values directly passed
 test_variadic_add_optional(1, 2, 3); // 1

tests/src/lib.rs

@@ -90,15 +90,11 @@ pub fn test_variadic_args(params: &[&Zval]) -> Vec<Zval> {
 
 #[php_function(optional = "numbers")]
 pub fn test_variadic_add_optional(number: u32, numbers: &[&Zval]) -> u32 {
-    println!("Optional numbers: {:?}", numbers);
-    // numbers is a slice of 4 Zvals all of type long
     number
 }
 
 #[php_function]
 pub fn test_variadic_add_required(numbers: &[&Zval]) -> Vec<Zval> {
-    // numbers is a slice of 4 Zvals all of type long
-    // *numbers[0].as_number()
     numbers.iter().map(|x| x.shallow_clone()).collect()
 }
 
@@ -107,17 +103,6 @@ pub fn test_variadic_all_types(everything: &[&Zval]) -> Vec<Zval> {
     everything.iter().map(|x| x.shallow_clone()).collect()
 }
 
-// Rust type &[&Zval] must be converted because to Vec<Zval> because
-// of lifetime hell...  #[php_function] does not support lifetime.
-// error1: error[E0261]: use of undeclared lifetime name `'a`
-// error2: lifetime `'a` is missing in item created through this procedural
-// macro #[php_function]
-// pub fn test_variadic_optional_args<'a>(params: &'a [&'a Zval]) -> &'a [&'a
-// Zval] {
-//     println!("Params: {:#?}", params);
-//     params
-// }
-
 #[php_class]
 pub struct TestClass {
     string: String,