diff --git a/language/tokens.ts b/language/tokens.ts index 78743cf..9ba0870 100644 --- a/language/tokens.ts +++ b/language/tokens.ts @@ -333,7 +333,7 @@ export function tokenise(statement: string, options: TokeniseOptions = {}): Toke if (state === ReadState.IN_STRING) { if (possibleEscape) { currentText += `''`; - i += 2; + i++; } else { currentText += statement[i]; result.push({ value: currentText, type: `string`, range: { start: startsAt, end: startsAt + currentText.length, line: lineNumber } }); diff --git a/tests/suite/linter.test.ts b/tests/suite/linter.test.ts index cce1088..c6d6d01 100644 --- a/tests/suite/linter.test.ts +++ b/tests/suite/linter.test.ts @@ -3319,5 +3319,74 @@ test('issue_353_indent_4', async () => { expect(errors.length).toBe(0); console.log(indentErrors); + expect(indentErrors.length).toBe(0); +}); + +test(`issue_353_indent_5`, async () => { + const lines = [ + `**FREE`, + `*in01 = 'yy' in %list('zz': ' ''');`, + `*inLR = *ON;`, + `return;`, + `dcl-proc test;`, + ` // If the line ends with a known end-of line character, then`, + ` // assume it is free format. Make sure to strip any free-format`, + ` // comment first. The stripping is not precise and will cause`, + ` // a free form line that has a '//' literal to not be detected.`, + ` eol = ';';`, + `end-proc;` + ].join(`\n`); + + const cache = await parser.getDocs(uri, lines, { ignoreCache: true, withIncludes: false }); + + const { indentErrors, errors } = Linter.getErrors({ uri, content: lines }, { + indent: 2 + }, cache); + + expect(errors.length).toBe(0); + expect(indentErrors.length).toBe(0); +}); + +test(`issue_353_indent_6`, async () => { + const lines = [ + `**FREE`, + `*in01 = 'yy' in %list('zz': ' ''');`, + `*inLR = *ON;`, + `return;`, + ``, + `dcl-proc test;`, + ` if specType = 'C' and not likelyComment;`, + ` if not (%subst(srcDta: 7: 2) in`, + ` %list(' ': 'L0': 'L1': 'L2': 'L3': 'L4': 'L5'`, + ` : 'L6': 'L7': 'L8': 'L9': 'LR': 'SR': 'AN': 'OR'));`, + ` return *ON;`, + ` endif;`, + ` endif;`, + ``, + ` // If the line ends with a known end-of line character, then`, + ` // assume it is free format. Make sure to strip any free-format`, + ` // comment first. The stripping is not precise and will cause`, + ` // a free form line that has a '//' literal to not be detected.`, + ` eol = ';';`, + ` posStr = %scan('//': srcDta);`, + ` if posStr > 1;`, + ` evalr eol = %trim(%subst(srcDta: 1: posStr - 1));`, + ` endif;`, + ` if (eol in %list(';': ':': '+': '-': '/': '*': '=': '_'));`, + ` return *ON;`, + ` endif;`, + ``, + ` // Despite our best guesses, we cannot confirm this to be a`, + ` // free format line, so indicate that it is _not_ free format.`, + ` return *OFF;`, + `end-proc;`, + ].join(`\n`); + + const cache = await parser.getDocs(uri, lines, { ignoreCache: true, withIncludes: false }); + + const { indentErrors, errors } = Linter.getErrors({ uri, content: lines }, { + indent: 2 + }, cache); + expect(indentErrors.length).toBe(0); }); \ No newline at end of file