- User Since
- Feb 24 2017, 6:42 PM (141 w, 5 d)
Sun, Oct 27
I agree that conceptually this should be moved to (or supplemented by) the actual language plugins, but also agree that this might be out of scope for this change.
As far as the PHP changes are concerned this is fine with me. However, I'd prefer those patterns to be unit tested and not just documented in comments.
Could you have a look at adding those tests?
Conceptually, this looks fine. However, like this we end up with two different descriptions of "Array of" types in the navigation popups.
While this shows integer arrays as int, for variadics it will show array of (int). I think at the very least both should be displayed the same.
Oct 7 2019
Mar 18 2019
Jan 9 2019
I'm afraid I don't know much about this either :-/
The fact that the unit tests are currently completely broken doesn't help. I did try them with and without this change and what I *can* say is that it's neither worse nor better.
Jan 5 2019
Nov 22 2018
Nov 17 2018
Oct 28 2018
Add more checks to the if-statements
Restructure if-statements in visitStaticMember.
Remove commented code from unit tests.
Oct 27 2018
Sep 5 2018
Aug 19 2018
Aug 17 2018
Split out duplicate code and reordered it a bit to make it easier to follow.
Splitting some code out into a separate function is a bit more tricky than I thought. I have prototype code for it, but it just doesn't quite work right (unit tests don't pass :( ).
I'll merge it for now the way it is and have a look at the refactored version after.
Aug 16 2018
Added more unit tests for bugs fixed by this :)
Aug 14 2018
Aug 13 2018
Shows as "Accepted" here, so lot's see :)
Compare canonical paths.
Proper fix in 45329c939edfcb3d57d6a53b83ca435da5aa04d8
Aug 11 2018
If there's no objections, I'm gonna merge this on Monday together with D13580
Aug 2 2018
Less spaghetti in the lexer.