From 6a55ca346e543e4e112648cca2a01230c32b21ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Valim?= <jose.valim@gmail.com>
Date: Thu, 30 Sep 2010 11:40:51 +0200
Subject: [PATCH] Revert "removing crazy finalizer code until there is proof
 that we need it"

This reverts commit a40e3c1a9604ab3737ad2465c8f6a6db0fe0cc78.
---
 actionpack/lib/action_view/template.rb | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/actionpack/lib/action_view/template.rb b/actionpack/lib/action_view/template.rb
index 405e1736baf..164d177dcc1 100644
--- a/actionpack/lib/action_view/template.rb
+++ b/actionpack/lib/action_view/template.rb
@@ -101,6 +101,16 @@ module ActionView
     attr_reader :source, :identifier, :handler, :virtual_path, :formats,
                 :original_encoding
 
+    # This finalizer is needed (and exactly with a proc inside another proc)
+    # otherwise templates leak in development.
+    Finalizer = proc do |method_name, mod|
+      proc do
+        mod.module_eval do
+          remove_possible_method method_name
+        end
+      end
+    end
+
     def initialize(source, identifier, handler, details)
       @source             = source
       @identifier         = identifier
@@ -245,6 +255,7 @@ module ActionView
 
         begin
           mod.module_eval(source, identifier, 0)
+          ObjectSpace.define_finalizer(self, Finalizer[method_name, mod])
 
           method_name
         rescue Exception => e # errors from template code