Suggestion to avoid repeating `compile`

[Sergey Trofimov]

Looks like it is better move .compile out of (fn) scope to avoid this call on every render.
This call is quite fast but useless.

mustache.clojure/src/mustache/core.clj

Lines 24 to 35 in 5d28f63

 (fn [data] 
   (let [^MustacheFactory template-factory @mustache-factory 
         ^Mustache template (.compile template-factory template-name) 
         ^StringWriter buffer (StringWriter.)] 
     (.execute template buffer data) 
     (.toString buffer))) 
 (fn [data] 
   (let [^MustacheFactory template-factory mustache-factory 
         ^Mustache template (.compile template-factory template-name) 
         ^StringWriter buffer (StringWriter.)] 
     (.execute template buffer data) 
     (.toString buffer))))) 

[Bill Robertson]

At first glance, I think you're right. While the call may be fast, it probably makes work for the garbage collector.

It has been a long time since I've looked at this code though. So I would like to take a look at things and make sure that this is right.