Skip to content

Commit

Permalink
Correct the non-printable regexp. Closes nodeca#64
Browse files Browse the repository at this point in the history
Dervus Grim committed Feb 15, 2013
1 parent ec38ce8 commit 4a02edc
Showing 4 changed files with 17 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/js-yaml/loader.js
Original file line number Diff line number Diff line change
@@ -107,7 +107,7 @@ HEXADECIMAL_ESCAPE_SEQUENCES[CHAR_SMALL_U] = 4;
HEXADECIMAL_ESCAPE_SEQUENCES[CHAR_CAPITAL_U] = 8;


var PATTERN_NON_PRINTABLE = /[\x00-\x09\x0B\x0C\x0E-\x1F\x7F-\x84\x86-\x9F\uD800-\uDFFF\uFFFE\uFFFF]/;
var PATTERN_NON_PRINTABLE = /[\x00-\x08\x0B\x0C\x0E-\x1F\x7F-\x84\x86-\x9F\uD800-\uDFFF\uFFFE\uFFFF]/;
var PATTERN_NON_ASCII_LINE_BREAKS = /[\x85\u2028\u2029]/;
var PATTERN_FLOW_INDICATORS = /[,\[\]\{\}]/;
var PATTERN_TAG_HANDLE = /^(?:!|!!|![a-z\-]+!)$/i;
1 change: 1 addition & 0 deletions test/issues.js
Original file line number Diff line number Diff line change
@@ -10,4 +10,5 @@ describe('Issues.', function () {
require('./issues/issue-33.js');
require('./issues/issue-46.js');
require('./issues/issue-54.js');
require('./issues/issue-64.js');
});
1 change: 1 addition & 0 deletions test/issues/data/issue-64.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
mapping: contains tab
14 changes: 14 additions & 0 deletions test/issues/issue-64.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
'use strict';
/*global it */


var assert = require('assert');

var yaml = require('../../lib/js-yaml');


it('#8: Wrong error message when yaml file contains tabs', function () {
assert.doesNotThrow(
function () { require('./data/issue-64.yml'); },
yaml.YAMLException);
});

0 comments on commit 4a02edc

Please sign in to comment.