Skip to content

Implement 8.6 deprecations - #23074

Merged
Girgias merged 12 commits into
php:masterfrom
Girgias:2026-08-8.6-deprecations
Aug 10, 2026
Merged

Implement 8.6 deprecations#23074
Girgias merged 12 commits into
php:masterfrom
Girgias:2026-08-8.6-deprecations

Conversation

@Girgias

@Girgias Girgias commented Aug 5, 2026

Copy link
Copy Markdown
Member

Comment thread ext/bz2/bz2_filter.c Outdated
Comment thread ext/mbstring/mbstring.c Outdated
Comment thread ext/standard/basic_functions.stub.php Outdated
Comment thread ext/standard/basic_functions.stub.php Outdated
@Girgias
Girgias force-pushed the 2026-08-8.6-deprecations branch from 96c961e to 95cd7c5 Compare August 6, 2026 17:28
throw new Exception($message);
});

/* Throwing error handlers do no apply for compile time deprecations */

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.

Does wrapping the function into if(random_int(1, 1)) work? Otherwise you could require __DIR__ . '/readonly_as_fn_name_is_deprecated.phpt'; to reuse the other test.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No, it doesn't do anything.

Comment thread ext/standard/tests/array/sort/locale_sort.phpt Outdated
Girgias added 8 commits August 9, 2026 20:58
@Girgias
Girgias force-pushed the 2026-08-8.6-deprecations branch from 95cd7c5 to b36f51d Compare August 9, 2026 20:37
@Girgias
Girgias force-pushed the 2026-08-8.6-deprecations branch from b36f51d to bca9332 Compare August 9, 2026 20:52
@Girgias
Girgias marked this pull request as ready for review August 9, 2026 23:00
@Girgias
Girgias merged commit e95647c into php:master Aug 10, 2026
18 checks passed
@Girgias
Girgias deleted the 2026-08-8.6-deprecations branch August 10, 2026 17:21
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.

3 participants