]> granicus.if.org Git - php/commitdiff
Revert gd and mysqli to former calls for arg count errors. Fix first identified issue...
authorAnthony Ferrara <ircmaxell@gmail.com>
Wed, 18 Mar 2015 20:14:02 +0000 (16:14 -0400)
committerAnthony Ferrara <ircmaxell@gmail.com>
Wed, 18 Mar 2015 20:14:02 +0000 (16:14 -0400)
Zend/tests/generators/generator_return_without_value.phpt
Zend/zend_compile.c
ext/gd/gd.c
ext/mysqli/mysqli_api.c

index c50d5916d66d0e7c9eb65d39add483863316e34e..dfea5aa78d5d0a39a21b6317e2183dc79bdbb8b6 100644 (file)
@@ -13,13 +13,31 @@ function gen2() {
     return null;
 }
 
+function gen3() {
+       return;
+    yield;
+}
+
+function gen4() {
+       return;
+    yield;
+}
+
 var_dump(gen());
 
 var_dump(gen2());
 
+var_dump(gen3());
+
+var_dump(gen4());
+
 ?>
 --EXPECTF--
 object(Generator)#%d (0) {
 }
 object(Generator)#%d (0) {
+}
+object(Generator)#%d (0) {
+}
+object(Generator)#%d (0) {
 }
\ No newline at end of file
index bfbb552f3f5cf16d2a37bf5c2d5d1eee22695f6c..1ec6a11c971aab314f62d29b78b49f7bf9609b31 100644 (file)
@@ -1931,7 +1931,7 @@ static void zend_emit_return_type_check(znode *expr, zend_arg_info *return_info)
 {
        if (return_info->type_hint != IS_UNDEF) {
                zend_op *opline = zend_emit_op(NULL, ZEND_VERIFY_RETURN_TYPE, expr, NULL);
-               opline->extended_value = (CG(declarables).strict_types ? 1 : 0);
+               opline->extended_value = CG(declarables).strict_types;
        }
 }
 /* }}} */
@@ -3402,7 +3402,7 @@ void zend_compile_return(zend_ast *ast) /* {{{ */
                opline->op1.var = CG(context).fast_call_var;
        }
 
-    // Generator return types are handled separately
+       /* Generator return types are handled separately */
        if (!(CG(active_op_array)->fn_flags & ZEND_ACC_GENERATOR) && CG(active_op_array)->fn_flags & ZEND_ACC_HAS_RETURN_TYPE) {
                zend_arg_info *arg_info = CG(active_op_array)->arg_info - 1;
 
index 94e250763460b4c9f03b2626af50c2afd57e0cb5..9c252f45fc4352b2f4ee62508c334a1998fd47d2 100644 (file)
@@ -3940,13 +3940,13 @@ static void php_imagettftext_common(INTERNAL_FUNCTION_PARAMETERS, int mode, int
 
        if (mode == TTFTEXT_BBOX) {
                if (argc < 4 || argc > ((extended) ? 5 : 4)) {
-                       WRONG_PARAM_COUNT;
+                       ZEND_WRONG_PARAM_COUNT();
                } else if (zend_parse_parameters(argc, "ddss|a", &ptsize, &angle, &fontname, &fontname_len, &str, &str_len, &EXT) == FAILURE) {
                        RETURN_FALSE;
                }
        } else {
                if (argc < 8 || argc > ((extended) ? 9 : 8)) {
-                       WRONG_PARAM_COUNT;
+                       ZEND_WRONG_PARAM_COUNT();
                } else if (zend_parse_parameters(argc, "rddlllss|a", &IM, &ptsize, &angle, &x, &y, &col, &fontname, &fontname_len, &str, &str_len, &EXT) == FAILURE) {
                        RETURN_FALSE;
                }
@@ -4395,7 +4395,7 @@ PHP_FUNCTION(imagepsbbox)
        BBox char_bbox, str_bbox = {0, 0, 0, 0};
 
        if (argc != 3 && argc != 6) {
-               WRONG_PARAM_COUNT;
+               ZEND_WRONG_PARAM_COUNT();
        }
 
        if (zend_parse_parameters(ZEND_NUM_ARGS(), "Srl|lld", &str, &fnt, &sz, &sp, &wd, &angle) == FAILURE) {
index 58c1132550d085fa174ddd188e125e00e2b8e8a8..eec5e74acb8fcc305c58eb73348542966a765ca2 100644 (file)
@@ -368,7 +368,7 @@ PHP_FUNCTION(mysqli_stmt_bind_param)
        args = safe_emalloc(argc, sizeof(zval), 0);
 
        if (zend_get_parameters_array_ex(argc, args) == FAILURE) {
-               WRONG_PARAM_COUNT;
+               zend_wrong_param_count();
                rc = 1;
        } else {
                rc = mysqli_stmt_bind_param_do_bind(stmt, argc, num_vars, args, start, types);