[BUG] lexer: line continuations with single line comments had duplica…

…ted new lines
Patitotective · Oct 13, 2022 · 0c41b40 · 0c41b40
1 parent 4ef2383
commit 0c41b40
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/kdl.nimble b/kdl.nimble
@@ -1,6 +1,6 @@
 # Package
 
-version       = "0.2.1"
+version       = "0.2.2"
 author        = "Patitotective"
 description   = "KDL document language Nim implementation"
 license       = "MIT"

diff --git a/src/kdl/lexer.nim b/src/kdl/lexer.nim
@@ -375,7 +375,7 @@ proc tokenSingleLineComment*() {.lexing: tkEmpty.} =
   lexer.consume 2
 
   while not lexer.eof(): # Consume until a new line or EOF
-    if lexer.tokenNewLine():
+    if lexer.tokenNewLine(addToStack = addToStack):
       break
     lexer.consume()
 
@@ -386,7 +386,7 @@ proc tokenLineCont*() {.lexing: tkLineCont.} =
   lexer.consume()
 
   lexer.skipwhitespaces()
-  if not lexer.tokenSingleLineComment() and not lexer.tokenNewLine(addToStack = false):
+  if not lexer.tokenSingleLineComment(addToStack = false) and not lexer.tokenNewLine(addToStack = false):
       lexer.error "Expected a new line"
 
 proc tokenLitMatches() {.lexing: tkEmpty.} = 

diff --git a/tests/test_cases/examples/website.kdl b/tests/test_cases/examples/website.kdl
@@ -6,7 +6,7 @@ html lang="en" {
         meta \
             name="description" \
             content="kdl is a document language, mostly based on SDLang, with xml-like semantics that looks like you're invoking a bunch of CLI commands!"
-        title "kdl - Kat's Document Language"
+        title "kdl - Kat's Document Language" // Comment
         link rel="stylesheet" href="/styles/global.css"
     }
     body {