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

Dead code removal is not removing loop statements #102

Closed
magalhas opened this issue May 20, 2015 · 2 comments
Closed

Dead code removal is not removing loop statements #102

magalhas opened this issue May 20, 2015 · 2 comments
Labels

Comments

@magalhas
Copy link
Contributor

for (;false;) {}
while(false) {}

The above statements are never removed by esmangle. Is this on purpose? If so, why?

@michaelficarra
Copy link
Member

Nope, that's a bug. Should be really easy to fix.

@magalhas
Copy link
Contributor Author

I'll create a PR in a couple of hours. Thank you.

Cumprimentos,
José Magalhães

On Thu, May 21, 2015 at 7:10 AM, Michael Ficarra notifications@github.com
wrote:

Nope, that's a bug. Should be really easy to fix.

Reply to this email directly or view it on GitHub:
#102 (comment)

michaelficarra added a commit that referenced this issue May 22, 2015
fix #102 unreachable branches removal was not targetting loop statements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants