-
Notifications
You must be signed in to change notification settings - Fork 40
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 C++ aggregator for sort
#349
base: future
Are you sure you want to change the base?
Conversation
Signed-off-by: Jan Bielak <[email protected]>
Signed-off-by: Jan Bielak <[email protected]>
Signed-off-by: Jan Bielak <[email protected]>
I also realized that for some reason |
Signed-off-by: Jan Bielak <[email protected]>
@nvasilakis Now the simple |
I updated the main comment with some notes and an option to-do list for better organization. |
Hi @janekb04 are you planning on completing this so that we merge it into mainline? |
I started implementing the C++ aggregator for FreeBSD that corresponds to
sort
.It doesn't support any flags and doesn't pass the unit test yet.It passes the basic unit test now (the version with no flags). The flags that are yet to be implemented:u
. Sort in unique moden
. Sort treating lines as numbers (remember about leading minus, decimal point or comma and thousands separator)r
. Reverse sort directionk[...]
. Sort by columnsf
.toupper
before sortingb
. Ignore whitespace at the beginning of a lined
. "consider only blanks and alphanumeric characters"There are also some other options, but they don't seem as easy to implement at first glance. All of the options above are also supported in FreeBSD and are the same. One thing that could be tricky is dealing with locales.