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

Comparing modifier types as equalities is broken. #27

Merged
merged 1 commit into from
Dec 15, 2015
Merged

Comparing modifier types as equalities is broken. #27

merged 1 commit into from
Dec 15, 2015

Conversation

gburtini
Copy link
Contributor

It doesn't capture "final private" which is admittedly a weird modifier. PHP Parser returns types as a bit set.

It doesn't capture "final private" which is admittedly a weird modifier. PHP Parser returns types as a bit set.
@gburtini gburtini changed the title Comparing types as equalities is broken. Comparing modifier types as equalities is broken. Dec 10, 2015
basilfx added a commit to basilfx/php-obfuscator that referenced this pull request Dec 15, 2015
@basilfx
Copy link

basilfx commented Dec 15, 2015

Can confirm it also works with the new PHP-Parser 2.0 library (which I incorporated into my fork).

See here for output.

naneau added a commit that referenced this pull request Dec 15, 2015
Comparing modifier types as equalities is broken.
@naneau naneau merged commit 4c17a08 into naneau:master Dec 15, 2015
@naneau
Copy link
Owner

naneau commented Dec 15, 2015

Thanks!

@naneau
Copy link
Owner

naneau commented Dec 15, 2015

I'll see about integrating 2.0 into this fork

@basilfx
Copy link

basilfx commented Dec 15, 2015

@naneau Integrating 2.0 is easy and only a few lines have to be changed.

You may want to check out my fork. I'm currently improving the scramblers. I use it on a 20k SLOC project using Phalcon, which works fine with ignoring only a few variables (including support for annotations, see #11).

@gburtini
Copy link
Contributor Author

Thanks guys. This is how open source should work!

@gburtini gburtini deleted the patch-1 branch December 15, 2015 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants