Skip to content

Commit d5dd9d5

Browse files
committed
Speed up test case
This test case did 100,000 includes of a non existing file to show the memory leak; this is not necessary, because after the first failing include, memory is not supposed to increase for further includes. Closes GH-7088.
1 parent a706d73 commit d5dd9d5

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

Zend/tests/bug79514.phpt

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,13 @@
22
Bug #79514 (Memory leaks while including unexistent file)
33
--FILE--
44
<?php
5-
$mem1 = memory_get_usage(true);
6-
for ($i = 0; $i < 100000; $i++) {
7-
@include __DIR__ . '/bug79514.doesnotexist';
8-
}
9-
$mem2 = memory_get_usage(true);
10-
var_dump($mem2 - $mem1 < 100000);
5+
$filename = __DIR__ . '/bug79514.doesnotexist';
6+
@include $filename;
7+
// Further include should not increase memory usage.
8+
$mem1 = memory_get_usage();
9+
@include $filename;
10+
$mem2 = memory_get_usage();
11+
var_dump($mem1 == $mem2);
1112
?>
1213
--EXPECT--
1314
bool(true)

0 commit comments

Comments
 (0)