Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Deprecate multiple default blocks in a switch#807

Closed
morrisonlevi wants to merge 1 commit into
php:PHP-5.6php/php-src:PHP-5.6from
morrisonlevi:deprecate_multiple_default_in_switchmorrisonlevi/php-src:deprecate_multiple_default_in_switchCopy head branch name to clipboard
Closed

Deprecate multiple default blocks in a switch#807
morrisonlevi wants to merge 1 commit into
php:PHP-5.6php/php-src:PHP-5.6from
morrisonlevi:deprecate_multiple_default_in_switchmorrisonlevi/php-src:deprecate_multiple_default_in_switchCopy head branch name to clipboard

Conversation

@morrisonlevi
Copy link
Copy Markdown
Contributor

Part of RFC: https://wiki.php.net/rfc/switch.default.multiple
This generates an E_DEPRECATED warning but continues to run the last default block. This change is intended for PHP 5.NEXT, not PHP 5.6.

@morrisonlevi morrisonlevi force-pushed the deprecate_multiple_default_in_switch branch from fe12a35 to cc15542 Compare September 5, 2014 21:30
@smalyshev smalyshev added the RFC label Nov 18, 2014
@nikic
Copy link
Copy Markdown
Member

nikic commented Feb 27, 2015

The PHP 7 error was merged in b476eca. This PR for the 5.7 deprecation can be closed as there is no 5.7 ^^

Please close it yourself, the QA pulls tool is broken again.

@php-pulls
Copy link
Copy Markdown

Comment on behalf of nikic at php.net:

Closing per previous comment.

@php-pulls php-pulls closed this Feb 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.