Allow linebreak/indent in destructured for variable (#5286)

* allow linebreak/indent in for variable pattern

* tests

* <

* condition

Author: helixbass

lib/coffeescript/lexer.js

@@ -182,7 +182,7 @@ exports.Lexer = class Lexer
       if tag is 'WHEN' and @tag() in LINE_BREAK
         tag = 'LEADING_WHEN'
       else if tag is 'FOR'
-        @seenFor = yes
+        @seenFor = {endsLength: @ends.length}
       else if tag is 'UNLESS'
         tag = 'IF'
       else if tag is 'IMPORT'
@@ -514,7 +514,7 @@ exports.Lexer = class Lexer
 
     prev = @prev()
     backslash = prev?[0] is '\\'
-    @seenFor = no unless backslash and @seenFor
+    @seenFor = no unless (backslash or @seenFor?.endsLength < @ends.length) and @seenFor
     @seenImport = no unless (backslash and @seenImport) or @importSpecifierList
     @seenExport = no unless (backslash and @seenExport) or @exportSpecifierList
 

src/lexer.coffee

@@ -576,3 +576,40 @@ test "#3778: Consistently always cache for loop range boundaries and steps, even
   a = 3; arrayEq [1, 2, 3], (for n in [1..+a]         then a = 4; n)
   a = 1; arrayEq [1, 2, 3], (for n in [1..3] by  a    then a = 4; n)
   a = 1; arrayEq [1, 2, 3], (for n in [1..3] by +a    then a = 4; n)
+
+test "for pattern variables can linebreak/indent", ->
+  listOfObjects = [a: 1]
+  sum = 0
+  for {
+    a
+    somethingElse
+    anotherProperty
+  } in listOfObjects
+    sum += a
+  eq a, 1
+
+  sum = 0
+  sum += a for {
+    a,
+    somethingElse,
+    anotherProperty,
+  } in listOfObjects
+  eq a, 1
+
+  listOfArrays = [[2]]
+  sum = 0
+  for [
+    a
+    nonexistentElement
+    anotherNonexistentElement
+  ] in listOfArrays
+    sum += a
+  eq a, 2
+
+  sum = 0
+  sum += a for [
+    a,
+    nonexistentElement,
+    anotherNonexistentElement
+  ] in listOfArrays
+  eq a, 2