cur_ip = p; should be used inside else statement because it delete previous value of cur_ip and may be nullptr when .c_str() will be executed
I dont understand why creating these issues over small code changes which are obviously correct. regardless feel free to open a pull request to fix this.
@elad335 , since we are not professionals, we need an approval from the genius, maybe optimization is restricted for some reason. Please don't be angry at us. We are just trying to be helpful.
Even if the fix was wrong or not you should always be encouraged to make a PR anyways hahah. That鈥檚 the whole reason why pull requests are a thing :p
Most helpful comment
@elad335 , since we are not professionals, we need an approval from the genius, maybe optimization is restricted for some reason. Please don't be angry at us. We are just trying to be helpful.