Skip to content

Commit

Permalink
Specify LINE_BREAKS on tokens with custom patterns (eclipse-langium…
Browse files Browse the repository at this point in the history
…#1685)

A warning was printed by Chevrotain for tokens using a custom matcher
function for the keyword pattern.
While this doesn't happen in the default implementation, it is possible
to override `buildKeywordPattern` and use a custom matcher function
  • Loading branch information
aabounegm authored Oct 8, 2024
1 parent fa9f405 commit 4bb3e78
Showing 1 changed file with 12 additions and 3 deletions.
15 changes: 12 additions & 3 deletions packages/langium/src/parser/token-builder.ts
Original file line number Diff line number Diff line change
Expand Up @@ -88,8 +88,10 @@ export class DefaultTokenBuilder implements TokenBuilder {
const tokenType: TokenType = {
name: terminal.name,
PATTERN: pattern,
LINE_BREAKS: true
};
if (typeof pattern === 'function') {
tokenType.LINE_BREAKS = true;
}
if (terminal.hidden) {
// Only skip tokens that are able to accept whitespace
tokenType.GROUP = isWhitespace(regex) ? Lexer.SKIPPED : 'hidden';
Expand Down Expand Up @@ -130,11 +132,18 @@ export class DefaultTokenBuilder implements TokenBuilder {
}

protected buildKeywordToken(keyword: Keyword, terminalTokens: TokenType[], caseInsensitive: boolean): TokenType {
return {
const keywordPattern = this.buildKeywordPattern(keyword, caseInsensitive);
const tokenType: TokenType = {
name: keyword.value,
PATTERN: this.buildKeywordPattern(keyword, caseInsensitive),
PATTERN: keywordPattern,
LONGER_ALT: this.findLongerAlt(keyword, terminalTokens)
};

if (typeof keywordPattern === 'function') {
tokenType.LINE_BREAKS = true;
}

return tokenType;
}

protected buildKeywordPattern(keyword: Keyword, caseInsensitive: boolean): TokenPattern {
Expand Down

0 comments on commit 4bb3e78

Please sign in to comment.