Skip to content
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

MethodReflection will be broken in PHP 7.3 #160

Closed
mvriel opened this issue Sep 23, 2018 · 1 comment
Labels

Comments

@mvriel
Copy link

@mvriel mvriel commented Sep 23, 2018

When I use the MethodReflection class's extractMethodContents in my unit tests on PHP 7.3 the following warning arises:

!!  In MethodReflection.php line 290:
!!                                                                                 
!!    Warning: "continue" targeting switch is equivalent to "break". Did you mean  
!!     to use "continue 2"?              
@Xerkus Xerkus added the duplicate label Sep 23, 2018
@Xerkus

This comment has been minimized.

Copy link
Member

@Xerkus Xerkus commented Sep 23, 2018

Fixed by #158 in release 3.3.1

@Xerkus Xerkus closed this Sep 23, 2018
sergeyklay added a commit to phalcon/zephir that referenced this issue Oct 14, 2019
dreamsxin added a commit to dreamsxin/zephir that referenced this issue Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.