From defc2f340378b4739cd715a7468c30b068db7793 Mon Sep 17 00:00:00 2001 From: kpdecker Date: Fri, 21 Oct 2011 08:34:24 -0500 Subject: [PATCH] Refactor deferred compile to use helper --- lib/handlebars/compiler/compiler.js | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/lib/handlebars/compiler/compiler.js b/lib/handlebars/compiler/compiler.js index d4ee32c0..7b759c14 100644 --- a/lib/handlebars/compiler/compiler.js +++ b/lib/handlebars/compiler/compiler.js @@ -753,16 +753,21 @@ Handlebars.precompile = function(string, options) { return new Handlebars.JavaScriptCompiler().compile(environment, options); }; -Handlebars.compile = function(string, compileOptions) { - var compiled; - compileOptions = compileOptions || {}; +Handlebars.compile = function(string, options) { + options = options || {}; + var compiled; + function compile() { + var ast = Handlebars.parse(string); + var environment = new Handlebars.Compiler().compile(ast, options); + var templateSpec = new Handlebars.JavaScriptCompiler().compile(environment, options, undefined, true); + return Handlebars.template(templateSpec); + } + + // Template is only compiled on first use and cached after that point. return function(context, options) { if (!compiled) { - var ast = Handlebars.parse(string); - var environment = new Handlebars.Compiler().compile(ast, options); - var templateSpec = new Handlebars.JavaScriptCompiler().compile(environment, options, undefined, true); - compiled = Handlebars.template(templateSpec); + compiled = compile(); } return compiled.call(this, context, options); };