Skip to content

Commit 314cd9c

Browse files
Use inline statements for variable declaration (#387)
* Clean indenting of generated code * Alpha ident generation * Use inline statements for variable declaration
1 parent b3992b1 commit 314cd9c

File tree

2 files changed

+7
-42
lines changed

2 files changed

+7
-42
lines changed

src/features/runtime-checks/script-overload.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,12 +92,7 @@ function stringifyScope (scope, scopePath) {
9292
if (value instanceof Map) {
9393
const proxyName = getOrGenerateIdentifier(['_proxyFor_', varOutName])
9494
output += `
95-
let ${proxyName}
96-
if (${scopePath.join('?.')}?.${key} === undefined) {
97-
${proxyName} = Object.bind(null);
98-
} else {
99-
${proxyName} = ${scopePath.join('.')}.${key};
100-
}
95+
let ${proxyName} = ${scopePath.join('?.')}?.${key} ? ${scopePath.join('.')}.${key} : Object.bind(null);
10196
`
10297
const keys = Array.from(value.keys())
10398
output += stringifyScope(value, [...scopePath, key])

unit-test/script-overload-snapshots/out/1.js

Lines changed: 6 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -21,38 +21,13 @@
2121
}
2222
})
2323
}
24-
let _ddg_b
25-
if (parentScope?.navigator === undefined) {
26-
_ddg_b = Object.bind(null);
27-
} else {
28-
_ddg_b = parentScope.navigator;
29-
}
24+
let _ddg_b = parentScope?.navigator ? parentScope.navigator : Object.bind(null);
3025
let _ddg_c = "testingThisOut";
31-
let _ddg_e
32-
if (parentScope?.navigator?.mediaSession === undefined) {
33-
_ddg_e = Object.bind(null);
34-
} else {
35-
_ddg_e = parentScope.navigator.mediaSession;
36-
}
26+
let _ddg_e = parentScope?.navigator?.mediaSession ? parentScope.navigator.mediaSession : Object.bind(null);
3727
let _ddg_f = "playing";
38-
let _ddg_h
39-
if (parentScope?.navigator?.mediaSession?.doesNotExist === undefined) {
40-
_ddg_h = Object.bind(null);
41-
} else {
42-
_ddg_h = parentScope.navigator.mediaSession.doesNotExist;
43-
}
44-
let _ddg_j
45-
if (parentScope?.navigator?.mediaSession?.doesNotExist?.depth === undefined) {
46-
_ddg_j = Object.bind(null);
47-
} else {
48-
_ddg_j = parentScope.navigator.mediaSession.doesNotExist.depth;
49-
}
50-
let _ddg_l
51-
if (parentScope?.navigator?.mediaSession?.doesNotExist?.depth?.a === undefined) {
52-
_ddg_l = Object.bind(null);
53-
} else {
54-
_ddg_l = parentScope.navigator.mediaSession.doesNotExist.depth.a;
55-
}
28+
let _ddg_h = parentScope?.navigator?.mediaSession?.doesNotExist ? parentScope.navigator.mediaSession.doesNotExist : Object.bind(null);
29+
let _ddg_j = parentScope?.navigator?.mediaSession?.doesNotExist?.depth ? parentScope.navigator.mediaSession.doesNotExist.depth : Object.bind(null);
30+
let _ddg_l = parentScope?.navigator?.mediaSession?.doesNotExist?.depth?.a ? parentScope.navigator.mediaSession.doesNotExist.depth.a : Object.bind(null);
5631
let _ddg_m = "boop";
5732
let _ddg_k = constructProxy(_ddg_l, {
5833
lot: _ddg_m
@@ -72,12 +47,7 @@
7247
mediaSession: _ddg_d
7348
});
7449
let navigator = _ddg_a;
75-
let _ddg_o
76-
if (parentScope?.document === undefined) {
77-
_ddg_o = Object.bind(null);
78-
} else {
79-
_ddg_o = parentScope.document;
80-
}
50+
let _ddg_o = parentScope?.document ? parentScope.document : Object.bind(null);
8151
let _ddg_p = "testingThisOut";
8252
let _ddg_n = constructProxy(_ddg_o, {
8353
cookie: _ddg_p

0 commit comments

Comments
 (0)