Skip to content
This repository was archived by the owner on Aug 5, 2022. It is now read-only.

Conversation

@zlw
Copy link

@zlw zlw commented Jun 3, 2012

I think it's nicer and cleaner way to do this.

@jpignata
Copy link

jpignata commented Jun 4, 2012

Definitely cleaner. I tested using a Regexp for each parameter too but found using #index to be faster in benchmarks. Perhaps there's yet another cleaner way to handle these parameters.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i used and instead of && so i don't have to add parentheses around method parameters (it's cleaner without them)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants