Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Removal of necessary parentheses #21

Closed
hasanyasin opened this Issue · 4 comments

2 participants

@hasanyasin

c||(c=393); // is processed correctly as:
c||(c=393); // parentheses kept.

However, in this situation:

c||(c=393,a=323,b=2321);

result is this:

c||c=393,a=323,b=2321;

causing a syntax error.

Version: slimit-0.5.5

@rspivak rspivak was assigned
@rspivak
Owner

Thanks for the bug report. I'll look into it soon.

@rspivak
Owner

Fixed in 0f446fd

To try the fix you'll have to install slimit from the master branch. Let me know if you have any other issues.

@hasanyasin

I pulled, tested, it works. Wow. You are fast!

I hope it did not break anything else. I will use slimit at Mushroom, so there will be a lot of testing, for sure.

Thank you.

@hasanyasin hasanyasin closed this
@rspivak
Owner

You're welcome and good luck with Mushroom.

@acatton acatton referenced this issue from a commit in acatton/slimit
@rspivak Fix rspivak#21
Removal of necessary parentheses
dfcbc62
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.