That's even nicer. Added to pull request.
Kenton White

On Nov 29, 2013, at 12:35 PM, Kasper Johansen <notifications@github.com> wrote:

> In include/active_support_cache_result.rb:
> 
> > @@ -0,0 +1,11 @@
> > +class GenderizeIoRb::ActiveSupportCacheResult < SimpleDelegator
> > +  VALID_ACCESSORS = [:result, :cache_db, :cache_as]
> > +  VALID_ACCESSORS.each do |name|
> > +    class_eval <<-ACCESSORS
> Have you considered avoiding eval? It is frowned upon in general.
> 
> Pretty sure you could do that like this here:
> 
> define_method(name) do
>   __send__("[]", name)
> end
> —
> Reply to this email directly or view it on GitHub.
>