Proper parenthesis for yield and await in conditional (#436)
Turns out, we need to always parenthesis when it's the first part of a conditional for both yield and await. Fixes #433
This commit is contained in:
1 parent
ae2fb73425
commit
5861c90913
3 files changed
+42
-7
No files matched your search
@@ -2,15 +2,41 @@ exports[`test conditional.js 1`] = `
|
||||
"function* f() {
|
||||
a = (yield) ? 1 : 1;
|
||||
a = yield 1 ? 1 : 1;
|
||||
a = (yield 1) ? 1 : 1;
|
||||
a = 1 ? yield : yield;
|
||||
a = 1 ? yield 1 : yield 1;
|
||||
}
|
||||
|
||||
function* f() {
|
||||
a = yield* 1 ? 1 : 1;
|
||||
a = (yield* 1) ? 1 : 1;
|
||||
a = 1 ? yield* 1 : yield* 1;
|
||||
}
|
||||
|
||||
async function f() {
|
||||
a = await 1 ? 1 : 1;
|
||||
a = (await 1) ? 1 : 1;
|
||||
a = 1 ? await 1 : await 1;
|
||||
}
|
||||
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
||||
function* f() {
|
||||
a = (yield) ? 1 : 1;
|
||||
a = yield 1 ? 1 : 1;
|
||||
a = (yield 1) ? 1 : 1;
|
||||
a = 1 ? yield : yield;
|
||||
a = 1 ? yield 1 : yield 1;
|
||||
}
|
||||
|
||||
function* f() {
|
||||
a = yield* 1 ? 1 : 1;
|
||||
a = (yield* 1) ? 1 : 1;
|
||||
a = 1 ? yield* 1 : yield* 1;
|
||||
}
|
||||
|
||||
async function f() {
|
||||
a = (await 1) ? 1 : 1;
|
||||
a = (await 1) ? 1 : 1;
|
||||
a = 1 ? await 1 : await 1;
|
||||
}
|
||||
"
|
||||
`;
|
||||
@@ -1,6 +1,19 @@
|
||||
function* f() {
|
||||
a = (yield) ? 1 : 1;
|
||||
a = yield 1 ? 1 : 1;
|
||||
a = (yield 1) ? 1 : 1;
|
||||
a = 1 ? yield : yield;
|
||||
a = 1 ? yield 1 : yield 1;
|
||||
}
|
||||
|
||||
function* f() {
|
||||
a = yield* 1 ? 1 : 1;
|
||||
a = (yield* 1) ? 1 : 1;
|
||||
a = 1 ? yield* 1 : yield* 1;
|
||||
}
|
||||
|
||||
async function f() {
|
||||
a = await 1 ? 1 : 1;
|
||||
a = (await 1) ? 1 : 1;
|
||||
a = 1 ? await 1 : await 1;
|
||||
}
|
||||
Reference in new issue
Block a user