Merge pull request #326 from atvise/fixed_variable_length_arrays

Fixed VLA definition because not every compiler supports that

Author: EmielBruijntjes

zend/parametersimpl.h

@@ -29,7 +29,7 @@ class ParametersImpl : public Parameters
         reserve(argc);
 
         // array to store all the arguments in
-        zval arguments[argc];
+        zval* arguments = static_cast<zval*>(alloca(argc * sizeof(zval)));
 
         // retrieve the arguments
         zend_get_parameters_array_ex(argc, arguments);

zend/value.cpp

@@ -903,7 +903,7 @@ Value Value::call(const char *name)
 Value Value::exec(int argc, Value *argv) const
 {
     // array of zvals to execute
-    zval params[argc];
+    zval* params = static_cast<zval*>(alloca(argc * sizeof(zval)));
 
     // convert all the values
     for(int i = 0; i < argc; i++) { params[i] = *argv[i]._val; }
@@ -925,7 +925,7 @@ Value Value::exec(const char *name, int argc, Value *argv) const
     Value method(name);
 
     // array of zvals to execute
-    zval params[argc];
+    zval* params = static_cast<zval*>(alloca(argc * sizeof(zval)));
 
     // convert all the values
     for(int i = 0; i < argc; i++) { params[i] = *argv[i]._val; }
@@ -947,7 +947,7 @@ Value Value::exec(const char *name, int argc, Value *argv)
     Value method(name);
 
     // array of zvals to execute
-    zval params[argc];
+    zval* params = static_cast<zval*>(alloca(argc * sizeof(zval)));
 
     // convert all the values
     for(int i = 0; i < argc; i++) { params[i] = *argv[i]._val; }