-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Furi: update string documentation #3803
Conversation
Compiled f7 firmware for commit |
I'll make use of this PR to raise a point - is there any practical purpose to Making it not destruct the source string (just empty it) may be more productive, as then it'll allow for patterns where e.g. a temporary string is being built in a loop, and results are being moved out to destinations after every step. In pseudocode, I mean something like: FuriString* tmp = furi_string_alloc();
while (results_not_done) {
// Do stuff to tmp
results.push_back(furi_string_alloc_move(tmp));
}
furi_string_free(tmp); |
Co-authored-by: hedger <hedger@nanode.su>
What's new
Verification
Checklist (For Reviewer)