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

Can't use modulus operator on the result of a function #857

Closed
ejdyksen opened this issue Jan 22, 2015 · 1 comment
Closed

Can't use modulus operator on the result of a function #857

ejdyksen opened this issue Jan 22, 2015 · 1 comment

Comments

@ejdyksen
Copy link

Here's the contents of my test sass file:

$list: "item-1" "item-2" "item-3";

#hello {
  @if length($list) % 2 == 0 {
    color: blue;
  }

  @else {
    color: red;
  }
}

This was working in node-sass 1.2.3 and is broken in 2.0.0-beta. So that means it was broken sometime between 5f3558d and cf7c1d1.

Background:
I'm getting a segfault when compiling Neat, using node-sass 2.0.0-beta. This line and this line test the even-ness of the length of a list using mod (%).

@xzyfer
Copy link
Contributor

xzyfer commented Jan 22, 2015

Confirmed this still results in a segmentation fault on the latest master.

This appears to be fixed on the latest master and will be in 3.2.

@xzyfer xzyfer added this to the 3.2 milestone Jan 22, 2015
@xzyfer xzyfer changed the title Can't use modulus operator on the result of the length() function Can't use modulus operator on the result of a function Feb 16, 2015
@xzyfer xzyfer self-assigned this Feb 16, 2015
@xzyfer xzyfer closed this as completed Feb 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants