Skip to content

ext/intl: Refactor ResourceBundle get and dimension access - #13503

Merged
Girgias merged 4 commits into
php:masterfrom
Girgias:intl-ressource-bundle
Feb 26, 2024
Merged

Girgias merged 4 commits into
php:masterfrom
Girgias:intl-ressource-bundle

Conversation

@Girgias

@Girgias Girgias commented Feb 25, 2024

Copy link
Copy Markdown
Member

So it turns out ResourceBundle's overloading of the dimension handlers is totally dumb...

Comment thread ext/intl/resourcebundle/resourcebundle_class.c Outdated
@Girgias
Girgias force-pushed the intl-ressource-bundle branch from b16f705 to c55719e Compare February 26, 2024 14:46
@Girgias
Girgias requested a review from devnexen February 26, 2024 14:47

@devnexen devnexen 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.

LCTM

@Girgias
Girgias merged commit dbf0b6a into php:master Feb 26, 2024
@Girgias
Girgias deleted the intl-ressource-bundle branch February 26, 2024 17:03
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