rfc:trailing-comma-function-args
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revisionNext revisionBoth sides next revision | ||
rfc:trailing-comma-function-args [2013/02/20 11:23] – Include declarations as well as calls pollita | rfc:trailing-comma-function-args [2013/03/25 18:28] – [Vote] pollita | ||
---|---|---|---|
Line 1: | Line 1: | ||
- | ====== Request for Comments: | + | ====== Request for Comments: |
* Version: 1.0 | * Version: 1.0 | ||
* Date: 2013-02-19 | * Date: 2013-02-19 | ||
* Author: Sara Golemon < | * Author: Sara Golemon < | ||
- | * Status: | + | * Status: |
* First Published at: http:// | * First Published at: http:// | ||
Line 44: | Line 44: | ||
===== The patch ===== | ===== The patch ===== | ||
- | Could not be much simpler... | + | https://github.com/sgolemon/php-src/compare/master...trailing-comma |
- | + | ||
- | diff --git a/Zend/zend_language_parser.y b/Zend/zend_language_parser.y | + | |
- | index ccbc9b1..6b6de2e 100644 | + | |
- | | + | |
- | +++ b/ | + | |
- | @@ -515,7 +515,7 @@ new_else_single: | + | |
- | + | ||
- | + | ||
- | | + | |
- | - | + | |
- | + | + | |
- | | /* empty */ | + | |
- | ; | + | |
- | + | ||
- | @@ -542,7 +542,7 @@ optional_class_type: | + | |
- | + | ||
- | | + | |
- | ' | + | |
- | | + | |
- | + | + | |
- | | ' | + | |
- | ; | + | |
- | + | ||
+ | HipHop version: https:// | ||
===== What's left out on purpose ===== | ===== What's left out on purpose ===== | ||
Line 91: | Line 69: | ||
} | } | ||
- | ===== Tests ===== | + | ===== Vote ===== |
+ | |||
+ | Note: Consideration for 5.6 inclusion, not 5.5. | ||
- | Yes, I'll write some. | + | <doodle |
+ | title=" | ||
+ | | ||
+ | * No | ||
+ | </ | ||
===== Changelog ===== | ===== Changelog ===== | ||
* Expanded proposal to include declarations as well as arguments 2013-02-20 11:23 GMT | * Expanded proposal to include declarations as well as arguments 2013-02-20 11:23 GMT |
rfc/trailing-comma-function-args.txt · Last modified: 2017/09/22 13:28 by 127.0.0.1