github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[jOOQ/jOOQ#18840] JSONParser::parseStringLiteral should omit StringBuilder copy if no escape sequence is found

Browse Source
This commit is contained in:
Lukas Eder 2025-08-07 14:24:24 +02:00
parent ae864549a4
commit 5f70242fcf
1 changed files with 19 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
jOOQ/src/main/java/org/jooq/impl/JSONParser.java
View File

@ -160,13 +160,21 @@ final class JSONParser extends AbstractParseContext {
private final String parseStringLiteral() {
parse('"', false);
StringBuilder sb = new StringBuilder();
StringBuilder sb = null;
int p0 = position();
int p1 = p0;
for (int i = position(); i < chars.length; i++) {
for (int i = p0; i < chars.length; i++) {
char c1 = character(i);
switch (c1) {
case '\\': {
if (sb == null) {
sb = new StringBuilder();
sb.append(chars, p0, i - p0);
}
else
sb.append(chars, p1, i - p1);
i++;
char c2 = character(i);
@ -200,17 +208,23 @@ final class JSONParser extends AbstractParseContext {
break;
}
p1 = i + 1;
sb.append(c1);
break;
}
case '"': {
position(i + 1);
parseWhitespaceIf();
return sb.toString();
if (sb != null) {
sb.append(chars, p1, i - p1);
return sb.toString();
}
else
return new String(chars, p0, i - p0);
}
}
sb.append(c1);
}
throw exception("String literal not terminated");