-
Notifications
You must be signed in to change notification settings - Fork 28
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
Optimization: Compare strings to a null string directly #168
Conversation
johnsonjh
commented
Nov 4, 2020
•
edited
Loading
edited
- Small run-time optimization
conflicts, not worth my time to fix, shouldn't really be worth your time either |
Not really any of my time at all, I believe these were automatically applied by some linter/checker or another. I normally run all the sources through a small battery of them, so, no real time of mine inovlved - leave open for now, and if it picks it up again, I'll submit rebased - but yeah, if I have to do it manually, it's such a small optimization that I won't bother. |
@cjdelisle autorebased |
|
|