refactor(core): Use Nullish coalescing assignment for view getters. (#56242)

JeanMeche · thePunderWoman · commit 33b0d25bfddf · 2024-06-03T20:10:35.000Z
Micro optim, this wasn't minified by the optimizer. PR Close #56242
diff --git a/packages/core/src/render3/instructions/shared.ts b/packages/core/src/render3/instructions/shared.ts
@@ -1955,11 +1955,11 @@ export function storePropertyBindingMetadata(
 
 export function getOrCreateLViewCleanup(view: LView): any[] {
   // top level variables should not be exported for performance reasons (PERF_NOTES.md)
-  return view[CLEANUP] || (view[CLEANUP] = []);
+  return (view[CLEANUP] ??= []);
 }
 
 export function getOrCreateTViewCleanup(tView: TView): any[] {
-  return tView.cleanup || (tView.cleanup = []);
+  return (tView.cleanup ??= []);
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -1955,11 +1955,11 @@ export function storePropertyBindingMetadata(`
`1955`	`1955`
`1956`	`1956`	`export function getOrCreateLViewCleanup(view: LView): any[] {`
`1957`	`1957`	`// top level variables should not be exported for performance reasons (PERF_NOTES.md)`
`1958`		`- return view[CLEANUP] \|\| (view[CLEANUP] = []);`
	`1958`	`+ return (view[CLEANUP] ??= []);`
`1959`	`1959`	`}`
`1960`	`1960`
`1961`	`1961`	`export function getOrCreateTViewCleanup(tView: TView): any[] {`
`1962`		`- return tView.cleanup \|\| (tView.cleanup = []);`
	`1962`	`+ return (tView.cleanup ??= []);`
`1963`	`1963`	`}`
`1964`	`1964`
`1965`	`1965`	`/**`