protobuf
protobuf copied to clipboard
Allow PHP reserved words as valid constant names
As of PHP 7.0, all the reserved words listed in kReservedNames are now allowed (and can also be added to kValidConstantNames) with the exception of "class". Although it'd probably be better to refactor the list to be kReservedConstantNames with just "class".
As the latest version of protobuf for PHP now supports a minimum of 7.0, we can open up these words. As changing these would break backwards compatibility, it would be better to target a future major version of protobuf for PHP with this change.
We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please add a comment.
This issue is labeled inactive because the last activity was over 90 days ago.
We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please reopen it.
This issue was closed and archived because there has been no new activity in the 14 days since the inactive label was added.
This is still a valid (and important) change that should be made
Reopened. Please feel free to submit a PR for this.
We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please add a comment.
This issue is labeled inactive because the last activity was over 90 days ago.
We triage inactive PRs and issues in order to make it easier to find active work. If this issue should remain active or becomes active again, please reopen it.
This issue was closed and archived because there has been no new activity in the 14 days since the inactive label was added.