forked from php/php-src
-
Notifications
You must be signed in to change notification settings - Fork 0
Rfc: inner class3 #4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
withinboredom
wants to merge
65
commits into
master
Choose a base branch
from
rfc/inner-class3
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 61 commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
bee945b
add new token
withinboredom 3d05dd7
create the new grammar
withinboredom a815b8b
add new ast pieces
withinboredom 978be06
handle modifiers
withinboredom 466f6c8
handle compiling class declarations
withinboredom cbfc5bc
add initial implementation;
withinboredom d9e9ccf
get return types working
withinboredom aa6d3e9
make ::class work
withinboredom bb94414
fix failing test
withinboredom b2581f4
add another class
withinboredom 787dd8e
add more tests to validate scope resolution keywords
withinboredom 05225e3
update reflection
withinboredom 6e8031c
enable visibility
withinboredom d4a4d32
add tests for autoloading
withinboredom b42316c
properly persist classes in opcache
withinboredom cbd51f7
handle return type visibility enforcement
withinboredom a86c26e
fix static member access
withinboredom 269a403
move tests
withinboredom a624033
add more tests and fix access modifiers
withinboredom e8d7570
add support for lexical scope
withinboredom 2e1d287
handle type visibility
withinboredom d33cd2f
fix opcache
withinboredom a7364af
fix test
withinboredom 63a0882
refine some error messages
withinboredom aafbda4
add type check to return type
withinboredom 0966e30
fix tests
withinboredom 6774e61
temporarily fix this test -- might want to undo this
withinboredom 9297a01
clean up a bit and check readonly
withinboredom 441eb93
handle visibility of methods
withinboredom bcee5ab
just do not cache -- more trouble than it is worth right now
withinboredom 4b847cd
properly handle lexical scope
withinboredom 5a5e44b
handle long names
withinboredom f24caa3
handle more visibility
withinboredom 20b2699
handle constructor visibility
withinboredom e4d3e62
prevent static access
withinboredom 609ce36
remove T_INNER_REF
withinboredom b2bb823
allow inner classes to use \\ now
withinboredom e17d677
fix prototype issue
withinboredom 63745a5
defer compiling inner classes
withinboredom 7a16a09
handle nesting classes in functions
withinboredom 1f2ddb2
update tests
withinboredom 4f441cb
do not elide return checks for inner classes
withinboredom d9ae53b
do visibility check during instantiation
withinboredom 864c4c6
replace wrong token
withinboredom c342b5d
remove inner class AST
withinboredom 6c28874
defer compiling inner classes
withinboredom a92ba63
allow looking up classes in scopes
withinboredom bbc0fed
fix isInnerClass
withinboredom ce0db15
add build artifacts
withinboredom e716cc9
Revert "defer compiling inner classes"
withinboredom a3abaff
update and fix tests
withinboredom 28a5466
handle namespaces properly
withinboredom ab0dd57
add to windows build
withinboredom 7ca157c
fix memory leak
withinboredom 416bde2
fixup last commit
withinboredom 1143b6c
ensure global scope does not have a lexical scope
withinboredom 6f10bda
ensure we do not escape class scope boundaries
withinboredom 226adc4
move namespace destruction to end of process
withinboredom 4ac9b82
remove dangling grammar
withinboredom 0ccd215
make a copy of the empty string
withinboredom 9579c43
ensure namespaces are not used after free
withinboredom 0a68147
tighten up namespaces
withinboredom a65d578
finally fix the elusive memory leak
withinboredom 7275681
stupid errors
withinboredom 13f130e
destroy namespaces after executor shutdown
withinboredom File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.