Skip to content

Implement "Followup improvements for ext/uri" RFC - URI type detection#22075

Merged
kocsismate merged 3 commits into
php:masterfrom
kocsismate:uri-followup1
May 24, 2026
Merged

Implement "Followup improvements for ext/uri" RFC - URI type detection#22075
kocsismate merged 3 commits into
php:masterfrom
kocsismate:uri-followup1

Conversation

@kocsismate
Copy link
Copy Markdown
Member

Comment thread ext/lexbor/lexbor/url/url.c Outdated
Comment thread ext/uri/uri_parser_rfc3986.c Outdated
Comment thread ext/uri/uri_parser_rfc3986.c Outdated
Comment thread ext/uri/uri_parser_rfc3986.h Outdated
Copy link
Copy Markdown
Member

@TimWolla TimWolla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but the lexbor changes need to be extracted first.

@kocsismate
Copy link
Copy Markdown
Member Author

LGTM, but the lexbor changes need to be extracted first.

@TimWolla Please review this again: #22093

kocsismate added a commit that referenced this pull request May 24, 2026
It's needed to implement #22075 for https://wiki.php.net/rfc/uri_followup#uri_type_detection.

Since the upstream changes to lexbor (lexbor/lexbor#362) haven't been released yet, a patch is added for now.
@TimWolla
Copy link
Copy Markdown
Member

Don't forget NEWS/UPGRADING.

@kocsismate kocsismate merged commit 1bf64ce into php:master May 24, 2026
1 check passed
@kocsismate kocsismate deleted the uri-followup1 branch May 24, 2026 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants