root/regexper-static
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Adding check that charset range is in correct order

Browse Source
This commit is contained in:
Jeff Avallone 2014-12-22 15:18:49 -05:00
parent 9b8eef9c02
commit d882ee8e08
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
spec/parser/javascript/charset_range_spec.js
View File

@ -17,6 +17,13 @@ describe('parser/javascript/charset_range.js', function() {
})); }));
}); });
it('throws an exception when the range is out of order', function() {
var parser = new javascript.Parser('z-a');
expect(() => {
parser.__consume__charset_range();
}).toThrow('Range out of order in character class: z-a');
});
describe('#_render', function() { describe('#_render', function() {
beforeEach(function() { beforeEach(function() {

4
src/js/parser/javascript/charset_range.js
View File

@ -26,5 +26,9 @@ export default {
setup() { setup() {
this.first = this.properties.first; this.first = this.properties.first;
this.last = this.properties.last; this.last = this.properties.last;
if (this.first.ordinal > this.last.ordinal) {
throw `Range out of order in character class: ${this.textValue}`;
}
} }
}; };