Skip to content

Stdlib: memory_get_usage() real_usage named parameter (#11578) - #11583

Merged
PurHur merged 1 commit into
masterfrom
agent/issue-11578-memory-get-usage-named
Jun 25, 2026
Merged

Stdlib: memory_get_usage() real_usage named parameter (#11578)#11583
PurHur merged 1 commit into
masterfrom
agent/issue-11578-memory-get-usage-named

Conversation

@PurHur

@PurHur PurHur commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Register real_usage in BuiltinParamNames for memory_get_usage() and memory_get_peak_usage() per php-src ext/standard/basic_functions.c stub.
  • Add maintainer repro and unit test.

Closes #11578

php-src reference

  • ext/standard/basic_functions.c — zif_memory_get_usage / zif_memory_get_peak_usage
  • Public stub: function memory_get_usage(bool $real_usage = false): int

Test plan

php test/repro/maintainer_gap_memory_get_usage_named_real_usage.php
# memory_get_usage_named_ok=1
# memory_get_peak_usage_named_ok=1

php bin/vm.php test/repro/maintainer_gap_memory_get_usage_named_real_usage.php
# memory_get_usage_named_ok=1
# memory_get_peak_usage_named_ok=1

php bin/compile.php -l test/repro/maintainer_gap_memory_get_usage_named_real_usage.php
# exit 0

./vendor/bin/phpunit test/unit/BuiltinParamNamesAliasTest.php --filter testMemoryGetUsageNamed
# OK

Made with Cursor

@PurHur @cursoragent
Register real_usage named parameter for memory_get_usage builtins (#1…
9dc51f4 
…1578)

Zend accepts memory_get_usage(real_usage: true); VM rejected unknown named
parameters because BuiltinParamNames lacked the php-src stub name.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit 587e1b1 into master Jun 25, 2026
@PurHur
PurHur deleted the agent/issue-11578-memory-get-usage-named branch June 25, 2026 10:01
@PurHur PurHur mentioned this pull request Jun 25, 2026
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Stdlib: memory_get_usage() real_usage: named — Unknown named parameter vs Zend (ext/standard/basic_functions.c)

1 participant

@PurHur