rfc:class_const_visibility
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revision | ||
rfc:class_const_visibility [2015/09/17 18:39] – Removing Final/Update BC for Reflection API sean-der | rfc:class_const_visibility [2017/09/22 13:28] (current) – external edit 127.0.0.1 | ||
---|---|---|---|
Line 3: | Line 3: | ||
* Date: 2015-09-13 | * Date: 2015-09-13 | ||
* Author: Sean DuBois < | * Author: Sean DuBois < | ||
- | * Status: | + | * Status: |
* First Published at: http:// | * First Published at: http:// | ||
Line 14: | Line 14: | ||
In a thread on php-internals a couple real world examples were offered. | In a thread on php-internals a couple real world examples were offered. | ||
- | | + | * Defining bitmasks/ |
- | | + | * Help make it more clear what is important, exposing harmless constants clutters documentation |
===== Proposal ===== | ===== Proposal ===== | ||
Line 23: | Line 23: | ||
Class constant may be define as public, private or protected. class constants declared | Class constant may be define as public, private or protected. class constants declared | ||
- | without any explict | + | without any explicit |
Proposed syntax: | Proposed syntax: | ||
Line 35: | Line 35: | ||
// Constants then also can have a defined visibility | // Constants then also can have a defined visibility | ||
- | private PRIVATE_CONST = 0; | + | private |
- | protected PROTECTED_CONST = 0; | + | protected |
- | public PUBLIC_CONST_TWO = 0; | + | public |
+ | |||
+ | //Constants can only have one visibility declaration list | ||
+ | private const FOO = 1, BAR = 2; | ||
} | } | ||
- | // | + | // |
interface ICache { | interface ICache { | ||
public const PUBLIC = 0; | public const PUBLIC = 0; | ||
- | protected | + | |
+ | } | ||
- | public function get($k, $v); | + | // |
- | public function set($k); | + | class testClass |
+ | const TEST_CONST = ' | ||
} | } | ||
+ | |||
+ | $obj = new ReflectionClass( " | ||
+ | $const = $obj-> | ||
+ | $consts = $obj-> | ||
</ | </ | ||
Line 54: | Line 63: | ||
===== Backward Incompatible Changes ===== | ===== Backward Incompatible Changes ===== | ||
- | |||
- | ==== Reflection Ext ==== | ||
- | The reflection extension will have to be updated to expose constants like properties. | ||
- | Instead of just an array of values there will be a dedicated ReflectionClassConstant class. | ||
===== Proposed PHP Version(s) ===== | ===== Proposed PHP Version(s) ===== | ||
Line 74: | Line 79: | ||
There will be new _ex APIs that allow callers to explicitly pass flags. | There will be new _ex APIs that allow callers to explicitly pass flags. | ||
- | If a extension accesses non-public structures (the class_constants HashTable) there will be breakage | + | If a extension accesses non-public structures (the now non-existent |
==== To Opcache ==== | ==== To Opcache ==== | ||
Line 80: | Line 85: | ||
Need update. | Need update. | ||
- | ===== Proposed Voting Choices | + | ===== Vote ===== |
Simple Yes/No option. This requires a 2/3 majority. | Simple Yes/No option. This requires a 2/3 majority. | ||
+ | |||
+ | This vote will close on 06:00 UTC on Tuesday 2015-10-27 | ||
+ | |||
+ | <doodle title=" | ||
+ | * Yes | ||
+ | * No | ||
+ | </ | ||
===== Patches and Tests ===== | ===== Patches and Tests ===== | ||
- | A pull request has been made, but not complete. It needs review, and help with opcache changes : https:// | + | A pull request has been made. It is feature complete but needs review, more tests, and help with opcache changes : https:// |
+ | This feature was merged into PHP master here: https:// | ||
===== References ===== | ===== References ===== | ||
Line 97: | Line 110: | ||
* V0.1 Initial version | * V0.1 Initial version | ||
* V0.2 Adopted by Sean DuBois < | * V0.2 Adopted by Sean DuBois < | ||
+ | * V0.2 Implemented |
rfc/class_const_visibility.txt · Last modified: 2017/09/22 13:28 by 127.0.0.1