We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I hesitated opening this as it's super minor, but is it intentional that there is no Parser::Source::Range#eql? (and #hash)?
Parser::Source::Range#eql?
#hash
This prevents using Ranges as meaningful hash keys or when conditions.
Range
when
I noticed this when wondering if RuboCop should optimize this O(n^2) algorithm although it would probably be completely negligible in that case compared to the actual policing...
O(n^2)
I'd gladly provide a PR if desired.
The text was updated successfully, but these errors were encountered:
Sorry, I missed this issue initially. Yes, it makes sense to me. Feel free to send a RP if you still need it.
Sorry, something went wrong.
+ Add Source::Range#eql? and hash [whitequark#670]
f90bb72
d315cbd
implemented in #675
No branches or pull requests
I hesitated opening this as it's super minor, but is it intentional that there is no
Parser::Source::Range#eql?
(and#hash
)?This prevents using
Range
s as meaningful hash keys orwhen
conditions.I noticed this when wondering if RuboCop should optimize this
O(n^2)
algorithm although it would probably be completely negligible in that case compared to the actual policing...I'd gladly provide a PR if desired.
The text was updated successfully, but these errors were encountered: