cleanup new parser handling
This commit is contained in:
parent
d125d57b3a
commit
9de1e8feab
2 changed files with 14 additions and 22 deletions
|
@ -14,32 +14,25 @@ export const parse = (source: string): Node => {
|
||||||
// });
|
// });
|
||||||
};
|
};
|
||||||
|
|
||||||
export const parse_expression_at = (source: string, index: number) => {
|
export const parse_expression_at = (source: string, index: number): number => {
|
||||||
// TODO: Clean up after acorn -> @babel/parser move
|
// TODO: Clean up after acorn -> @babel/parser move
|
||||||
try {
|
try {
|
||||||
|
// First, try to parse the expression. Unlike acorn, @babel/parser isn't relaxed
|
||||||
|
// enough to just stop after the first expression, so we almost always expect a
|
||||||
|
// parser error here instead. This is expected, so handle it.
|
||||||
parseExpression(source.slice(index), {
|
parseExpression(source.slice(index), {
|
||||||
sourceType: 'module',
|
sourceType: 'module',
|
||||||
plugins: ['jsx', 'typescript'],
|
plugins: ['jsx', 'typescript'],
|
||||||
});
|
});
|
||||||
throw new Error('Parse error.'); // Expected to fail.
|
throw new Error('Parse error.'); // Expected to fail.
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
if (!err.pos) {
|
if (err.message.startsWith('Unexpected token') && source[index + err.pos] === '}') {
|
||||||
throw err;
|
return index + err.pos;
|
||||||
}
|
}
|
||||||
try {
|
if (err.pos) {
|
||||||
const result = parseExpression(source.slice(index, index + err.pos), {
|
err.pos = index + err.pos;
|
||||||
sourceType: 'module',
|
|
||||||
plugins: ['jsx', 'typescript'],
|
|
||||||
});
|
|
||||||
result.start = index;
|
|
||||||
result.end = index + err.pos;
|
|
||||||
return result;
|
|
||||||
} catch (err2) {
|
|
||||||
if (err2.pos) {
|
|
||||||
err2.pos = index + err2.pos;
|
|
||||||
}
|
|
||||||
throw err2;
|
|
||||||
}
|
}
|
||||||
|
throw err;
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
// acornJsx.parseExpressionAt(source, index, {
|
// acornJsx.parseExpressionAt(source, index, {
|
||||||
|
|
|
@ -1,20 +1,19 @@
|
||||||
// @ts-nocheck
|
|
||||||
|
|
||||||
import { parse_expression_at } from '../acorn.js';
|
import { parse_expression_at } from '../acorn.js';
|
||||||
import { Parser } from '../index.js';
|
import { Parser } from '../index.js';
|
||||||
import { whitespace } from '../../utils/patterns.js';
|
import { whitespace } from '../../utils/patterns.js';
|
||||||
// import { Node } from 'estree';
|
// import { Node } from 'estree';
|
||||||
|
|
||||||
|
// @ts-ignore
|
||||||
export default function read_expression(parser: Parser): string {
|
export default function read_expression(parser: Parser): string {
|
||||||
try {
|
try {
|
||||||
const node = parse_expression_at(parser.template, parser.index);
|
const start = parser.index;
|
||||||
|
let index = parse_expression_at(parser.template, parser.index);
|
||||||
let num_parens = 0;
|
let num_parens = 0;
|
||||||
|
|
||||||
for (let i = parser.index; i < node.start; i += 1) {
|
for (let i = parser.index; i < start; i += 1) {
|
||||||
if (parser.template[i] === '(') num_parens += 1;
|
if (parser.template[i] === '(') num_parens += 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
let index = node.end;
|
|
||||||
while (num_parens > 0) {
|
while (num_parens > 0) {
|
||||||
const char = parser.template[index];
|
const char = parser.template[index];
|
||||||
|
|
||||||
|
@ -35,7 +34,7 @@ export default function read_expression(parser: Parser): string {
|
||||||
|
|
||||||
parser.index = index;
|
parser.index = index;
|
||||||
|
|
||||||
return parser.template.substring(node.start, node.end);
|
return parser.template.substring(start, index);
|
||||||
// return node as Node;
|
// return node as Node;
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
parser.acorn_error(err);
|
parser.acorn_error(err);
|
||||||
|
|
Loading…
Add table
Reference in a new issue