Skip to content

Commit de46fda

Browse files
committed
Update now outdated, incorrect comments in gh10801.phpt
Previously, the CTE did indeed not execute on function calls with named parameters. Now we can do this, so the comments need to be adjusted to not confuse the end user. The goal of the test remains the same though: we use different argument orders to check if there is a semantic difference.
1 parent caa9175 commit de46fda

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ext/opcache/tests/opt/gh10801.phpt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@ opcache.optimization_level=0xe0
88
opcache
99
--FILE--
1010
<?php
11-
// Named argument case and does not do CTE as expected
11+
// Named argument case with CTE
1212
print_r(array_keys(array: [1 => 1], strict: true, filter_value: 0));
13-
// Will not use named arguments and do CTE as expected
13+
// Will not use named arguments, and must result in the same output
1414
print_r(array_keys(array: [1 => 1], filter_value: 0, strict: true));
1515
?>
1616
--EXPECT--

0 commit comments

Comments
 (0)