Merge pull request #47 from goccy/feature/fix-literal-include-empty-line

Fix handling of literal token include empty line
goccy · Nov 19, 2019 · b24bdb8 · b24bdb8
2 parents debf487 + 4147f40
commit b24bdb8
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/parser/parser_test.go b/parser/parser_test.go
@@ -414,6 +414,24 @@ a:
     hoge
     fuga
     piyo
+`,
+		},
+		{
+			`
+a: |
+   bbbbbbb
+
+
+   ccccccc
+d: eeeeeeeeeeeeeeeee
+`,
+			`
+a: |
+   bbbbbbb
+
+
+   ccccccc
+d: eeeeeeeeeeeeeeeee
 `,
 		},
 	}

diff --git a/scanner/scanner.go b/scanner/scanner.go
@@ -102,6 +102,9 @@ func (s *Scanner) isNeededKeepPreviousIndentNum(ctx *Context, c rune) bool {
 }
 
 func (s *Scanner) updateIndent(ctx *Context, c rune) {
+	if s.isFirstCharAtLine && c == '\n' && ctx.isDocument() {
+		return
+	}
 	if s.isFirstCharAtLine && c == ' ' {
 		s.indentNum++
 		return
-Original file line number
+Diff line change
@@ Expand Up / @@ -414,6 +414,24 @@ a: @@
         hoge
         fuga
         piyo
+    `,
+    		},
+    		{
+    			`
+    a: |
+       bbbbbbb
+       ccccccc
+    d: eeeeeeeeeeeeeeeee
+    `,
+    			`
+    a: |
+       bbbbbbb
+       ccccccc
+    d: eeeeeeeeeeeeeeeee
     `,
     		},
     	}
@@ Expand Down @@