github/coffeescript
1
1
Fork 0
mirror of https://github.com/jashkenas/coffeescript.git synced 2026-05-03 03:00:14 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3783 from lydell/issue-3671

Browse Source 
Fix #3671: Allow step in optimized range comprehensions
This commit is contained in:
Michael Ficarra
2015-01-09 17:54:55 -08:00
parent e769423d52 a63009fccb
commit e0ad0d795d
4 changed files with 82 additions and 61 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lib/coffee-script/grammar.js
View File

@@ -421,6 +421,11 @@
return {
source: LOC(2)(new Value($2))
};
}), o('FOR Range BY Expression', function() {
return {
source: LOC(2)(new Value($2)),
step: $4
};
}), o('ForStart ForSource', function() {
$2.own = $1.own;
$2.name = $1[0];

124
lib/coffee-script/parser.js
View File

File diff suppressed because one or more lines are too long

3
src/grammar.coffee
View File

@@ -451,7 +451,8 @@ grammar =
]
ForBody: [
o 'FOR Range', -> source: LOC(2) new Value($2)
o 'FOR Range', -> source: (LOC(2) new Value($2))
o 'FOR Range BY Expression', -> source: (LOC(2) new Value($2)), step: $4
o 'ForStart ForSource', -> $2.own = $1.own; $2.name = $1[0]; $2.index = $1[1]; $2
]

11
test/comprehensions.coffee
View File

@@ -246,6 +246,17 @@ test "Optimized range comprehensions.", ->
ok exxes.join(' ') is 'x x x x x x x x x x'
test "#3671: Allow step in optimized range comprehensions.", ->
exxes = ('x' for [0...10] by 2)
eq exxes.join(' ') , 'x x x x x'
test "#3671: Disallow guard in optimized range comprehensions.", ->
throws -> CoffeeScript.compile "exxes = ('x' for [0...10] when a)"
test "Loop variables should be able to reference outer variables", ->
outer = 1
do ->