delegate PathSet's enumerator methods #8151

Merged
merged 1 commit into from Nov 9, 2012

4 participants

@nashby

No description provided.

@nikitug

Why not AS delegation here?

@nashby

@nikitug not sure if we need here something except Ruby Stdlib

@nikitug

@nashby yep, but maybe it's worth to keep the codebase more consistent. Take a look at ack 'delegate.*:to' in ActionPack.

@steveklabnik
Ruby on Rails member

We should use ActiveSupport's delegate here rather than Forwardable.

@steveklabnik
Ruby on Rails member

👍 :shipit:

@carlosantoniodasilva carlosantoniodasilva merged commit 6710f05 into rails:master Nov 9, 2012
@carlosantoniodasilva
Ruby on Rails member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment