mirrors/bun.sh
1
0
Fork 0
mirror of https://github.com/oven-sh/bun synced 2026-02-09 18:38:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

[JS Printer] Fix bug with template literals

Browse Source
This commit is contained in:
Jarred Sumner
2021-11-15 15:38:49 -08:00
parent 89a7e3bf2f
commit 1a60d7df98
3 changed files with 41 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
integration/scripts/snippets.json
View File

@@ -22,5 +22,6 @@
"/latin1-chars-in-regexp.js",
"/jsx-spacing.jsx",
"/jsx-entities.jsx",
"/optional-chain-with-function.js"
"/optional-chain-with-function.js",
"/template-literal.js"
]

37
integration/snippets/template-literal.js Normal file
View File

@@ -0,0 +1,37 @@
const css = (templ) => templ.toString();
const fooNoBracesUTF8 = css`
before
/* */
after
`;
const fooNoBracesUT16 = css`
before
🙃
after
`;
const fooUTF8 = css`
before
${true}
after
`;
const fooUTF16 = css`
before
🙃 ${true}
after
`;
export function test() {
for (let foo of [fooNoBracesUT16, fooNoBracesUTF8, fooUTF16, fooUTF8]) {
console.assert(
foo.includes("before"),
`Expected ${foo} to include "before"`
);
console.assert(foo.includes("after"), `Expected ${foo} to include "after"`);
}
return testDone(import.meta.url);
}

12
src/js_printer.zig
View File

@@ -1473,11 +1473,7 @@ pub fn NewPrinter(
p.print("`");
if (e.head.isPresent()) {
if (e.tag != null) {
p.print(e.head.utf8);
} else {
p.printStringContent(&e.head, '`');
}
p.printStringContent(&e.head, '`');
}
for (e.parts) |part| {
@@ -1485,11 +1481,7 @@ pub fn NewPrinter(
p.printExpr(part.value, .lowest, ExprFlag.None());
p.print("}");
if (part.tail.isPresent()) {
if (e.tag != null) {
p.print(part.tail.utf8);
} else {
p.printStringContent(&part.tail, '`');
}
p.printStringContent(&part.tail, '`');
}
}
p.print("`");