From 15bbc5ca1c5f809dc7a0bcf78a12086b9d0d0c35 Mon Sep 17 00:00:00 2001 From: Teymour Aldridge Date: Wed, 24 Jun 2020 17:41:28 +0100 Subject: [PATCH] Make two code items easier to understand: * Simplify an if statement * Also inlines a variable --- src/compiler/compile/render_dom/Renderer.ts | 3 +-- src/compiler/compile/render_dom/index.ts | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/compiler/compile/render_dom/Renderer.ts b/src/compiler/compile/render_dom/Renderer.ts index c02d646ebf..b38d941064 100644 --- a/src/compiler/compile/render_dom/Renderer.ts +++ b/src/compiler/compile/render_dom/Renderer.ts @@ -160,8 +160,7 @@ export default class Renderer { member.is_contextual = true; } else { member.is_non_contextual = true; - const variable = this.component.var_lookup.get(name); - member.variable = variable; + member.variable = this.component.var_lookup.get(name); } return member; diff --git a/src/compiler/compile/render_dom/index.ts b/src/compiler/compile/render_dom/index.ts index 3b5001d483..f1df051029 100644 --- a/src/compiler/compile/render_dom/index.ts +++ b/src/compiler/compile/render_dom/index.ts @@ -293,8 +293,7 @@ export default function dom( const variable = component.var_lookup.get(prop.name); if (variable.hoistable) return false; - if (prop.name[0] === '$') return false; - return true; + return prop.name[0] !== '$'; }); const reactive_stores = component.vars.filter(variable => variable.name[0] === '$' && variable.name[1] !== '$');