replace render-html with protocol

weavejester / hiccup

Fast library for rendering HTML in Clojure

http://weavejester.github.io/hiccup

Eclipse Public License 1.0

2.68k stars 174 forks source link

replace render-html with protocol #73

Closed gerrit-hntschl closed 11 years ago

gerrit-hntschl commented 11 years ago

Hi James,

during load-testing our hiccup based page we noticed a whole lot of lock contention for the multimethod lock of render-html. As the lock contention is only fixed in Clojure 1.5 (see here: http://dev.clojure.org/jira/browse/CLJ-988), I changed the multimethod to a protocol-based render-html that works just the same without the locking.

Gerrit

weavejester commented 11 years ago

Seems a reasonable change.