Added more utility functions for string manipulation in C++ #26
Conversation
✅ Deploy Preview for quicksnip ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| "#include <string>", | ||
| "#include <algorithm>", | ||
| "", | ||
| "std::string toUpperCase(const std::string& input) {", |
There was a problem hiding this comment.
Why are you taking the string by reference ?
There was a problem hiding this comment.
I think a constant reference is a good thing. It will not copy the whole variable, but just create a reference. But in this case yes, instead of making a const reference here and then creating another copy to return the string. They can just pass by value and return that string after transforming.
| "author": "tashifkhan" | ||
| }, | ||
| { | ||
| "title": "Find Substring", |
There was a problem hiding this comment.
Is this snippet really require, when its just a single function in the STL?
|
Hey, we recently updated how snippets are stored, please update your PR to match the changes, you will be able to see how to add snippets in |
|
Hey, changes were requested on your PR, but we got no news for more than a week, Closing this as stale. |
the functions include