source maps ignoring commented out lines

[philippkuehn]

Hey, i have a problem to get the source maps working.

@import "base/reset";

body {
  color: blue;
}

gives me for the body line 3 which is correct.

// @import "base/reset";

body {
  color: blue;
}

gives me for the body line 1 which is false.

It seems that all commented out lines are ignored in calculating the line numbers for the source map.

[philippkuehn]

This seems not to be fixed. I have the same problem as before.
And there is something more. This is strange.

Testcase 1:

body {
  color: blue;
}

.banner {
  color: blue;
}

Chrome 39:

body: line 1 -> correct
.banner: line 2 -> not correct (should be 5)

Firefox 34:

body: line 1 -> correct
.banner: line 5 -> correct

Testcase 2:

// @import "base/reset";

body {
  color: blue;
}

.banner {
  color: blue;
}

Chrome 39:

body: line 1 -> not correct (should be 3)
.banner: line 4 -> not correct (should be 7)

Firefox 34:

body: line 1 -> not correct (should be 3)
.banner: line 7 -> correct

[xzyfer]

Are you testing against master? This patch has not made it into an official release yet.

[philippkuehn]

yes. tested master.
(and i've updated my previous comment with more info)

[am11]

#324 and #605 are related, which are being tackled by @mgreter's #603 and #786 PRs.
To recap, the root-cause of this issue is not as simple as it seems.

[mgreter]

This issue should (hopefully) be addressed/fixed by #792.
Please try with latest master and open a new issue if the problem still exists.
Thank you!