AS::Callbacks: remove __define_runner

[bogdan]

I've suddenly realized that __define_runner method don't do anything right now: just define method that delegates work. Since this method is not part of the API - it can be omitted.

[josevalim]

Are you sure this doesn't break the test suite? I believe the runner is being used in some places...

[bogdan]

The need of runner came from the fact that previously caching was done only for keyed callback and there was a need to redefine runner for each set_callback call - it was dynamic. Now caching is done for everything. Runner method became static, that is why we don't need it and can call new _run_callbacks method directly that does the work(e.g. compilation caching).

-    def run_callbacks(kind, *args, &block)
-      send("_run_#{kind}_callbacks", *args, &block)
+    def run_callbacks(kind, key = nil, &block)
+      self.class.__run_callbacks(key, kind, self, &block)
     end

Anyway I've checked test suite for AP, AM and AR in order to ensure that everything is fine.

[josevalim]

Ok, merged, thanks!