Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NameError#missing_name? can just use NameError#name if the arg is a Symbol #18842

Merged
merged 1 commit into from
Feb 7, 2015

Conversation

yuki24
Copy link
Contributor

@yuki24 yuki24 commented Feb 7, 2015

NameError#name returns a missing name as a symbol. This means that if the given name is a symbol, it doesn't have to use #missing_name to get the last constant name in the error message.

…ymbol

NameError#name returns a missing name as a symbol, so if the given name
is a symbol, it doesn't have to use #missing_name to get the last constant
name in the error message.
rafaelfranca added a commit that referenced this pull request Feb 7, 2015
NameError#missing_name? can just use NameError#name if the arg is a Symbol
@rafaelfranca rafaelfranca merged commit bdeeca8 into rails:master Feb 7, 2015
@yuki24 yuki24 deleted the simplify-NameError#missing_name branch February 8, 2015 02:17
@yuki24
Copy link
Contributor Author

yuki24 commented Feb 8, 2015

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants