Merge pull request #304 from dplepage/fix-quote-stripping

Fix parsing of quotes followed by newlines.
nodeca · Oct 28, 2016 · 39c5f2c · 39c5f2c
2 parents 2bf232b + cb14c0b
commit 39c5f2c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/lib/js-yaml/loader.js b/lib/js-yaml/loader.js
@@ -532,8 +532,9 @@ function readSingleQuotedScalar(state, nodeIndent) {
       ch = state.input.charCodeAt(++state.position);
 
       if (ch === 0x27/* ' */) {
-        captureStart = captureEnd = state.position;
+        captureStart = state.position;
         state.position++;
+        captureEnd = state.position;
       } else {
         return true;
       }

diff --git a/test/issues/0303.js b/test/issues/0303.js
@@ -0,0 +1,11 @@
+'use strict';
+
+var assert = require('assert');
+var yaml   = require('../../');
+
+test('Loader should not strip quotes before newlines', function () {
+  var with_space = yaml.load("'''foo'' '");
+  var with_newline = yaml.load("'''foo''\n'");
+  assert.strictEqual(with_space, "'foo' ");
+  assert.strictEqual(with_newline, "'foo' ");
+});