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 threadext/lexbor/lexbor/url/url.c Outdated
Comment threadext/uri/uri_parser_rfc3986.c Outdated
Comment threadext/uri/uri_parser_rfc3986.c Outdated
Comment threadext/uri/uri_parser_rfc3986.h Outdated

@TimWollaTimWolla left a comment

Copy link
Copy Markdown
Member

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
MemberAuthor

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:masterMay 24, 2026
1 check passed
@kocsismate
kocsismate deleted the uri-followup1 branch May 24, 2026 21:35
Sign up for freeto 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

@kocsismate@TimWolla