-
Notifications
You must be signed in to change notification settings - Fork 355
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
Add various text-oriented wrapper libraries. #6988
base: develop
Are you sure you want to change the base?
Conversation
could the second half of this be simplified by using
or is there some advantage to using the unitized version? |
Per conversation with @belisarius222 ~2024.5.23, I have collected all of the strictly text-related libraries into one PR. These are explicitly NOT kelvin-versioned, and consist of wrappers, aliases, and simple helper functionality. @tinnus-napbus generating documentation should be straightforward as all of the functions are documented in each file, including examples. |
I guess /lib/bits is not technically part of this scope. |
Deadline for review is tomorrow, at which point if there are no comments we'll go ahead and merge. |
I have opened PR #7020 with optimisations for some of the functions in libstring & fixed a couple of functions that were incorrect (right-find, right-justify and arguably split which wasn't adding an empty string after a trailing separator). I will look at the other 2 quickly tomorrow. Sorry for not getting around to it earlier |
Optimise libstring and fix errors
ok I have opened PR #7021 with some changes to the other 2 libs (see PR for details) - not sure if you want the utf changes I made |
Add alternative functions for common text conversion operations.