Skip to content

Stdlib: stream wrapper parity — readfile/scandir/debug_zval_dump (#18417, #18418, #18419)#18422

Closed
PurHur wants to merge 3 commits into
masterfrom
agent/advanced-stdlib-stream-batch-18417-18419
Closed

Stdlib: stream wrapper parity — readfile/scandir/debug_zval_dump (#18417, #18418, #18419)#18422
PurHur wants to merge 3 commits into
masterfrom
agent/advanced-stdlib-stream-batch-18417-18419

Conversation

@PurHur

@PurHur PurHur commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

php-src refs: ext/standard/streams.c, ext/standard/dir.c, ext/standard/var.c.

Test plan

  • Issue repros (VM):
    ./script/docker-exec.sh -- bash -lc 'source script/php-env.sh
    php bin/vm.php test/repro/maintainer_gap_readfile_php_output_return.php
    php bin/vm.php test/repro/maintainer_gap_scandir_filter_wrapper_error.php
    php bin/vm.php test/repro/maintainer_gap_debug_zval_dump_stream.php'
    Output:
    -1 / 0 / 0
    scandir_warnings=2 (not implemented + errno 0 Success)
    opendir_warnings=1 (not implemented)
    resource(N) of type (stream) refcount(2)
    resource(N) of type (Unknown) refcount(2)
    
  • Compliance guards (VM + JIT):
    ./script/phpunit.sh --filter "readfile_php_output|scandir_filter_wrapper_error|debug_zval_dump_stream"
    # OK (6 tests, 6 assertions)
  • ./script/ci-fast.sh (inventory regen, spine 4705/4705, M5 fast gate)

Closes #18417
Closes #18418
Closes #18419

Made with Cursor

PurHur and others added 3 commits July 12, 2026 23:51
…ump (#18417, #18418, #18419)

Fix php://output readfile -1 sentinel, php:// wrapper dir-open Zend diagnostics,
and debug_zval_dump resource line format; add compliance guards and spine includes.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…verlap

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur

PurHur commented Jul 12, 2026

Copy link
Copy Markdown
Owner Author

Superseded: #18417/#18418/#18419 merged on master via #18421/#18420/#18423. Follow-up PR incoming for data:// dir-open + scandir errno followup parity.

@PurHur PurHur closed this Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant