GrayJack / coreutils

Core utils re-implementation for UNIX/UNIX-like systems written in Rust
Mozilla Public License 2.0
108 stars 40 forks source link

Sort: Adding long option --merge for -m flag, and some refactoring #132

Closed marcospb19 closed 4 years ago

marcospb19 commented 4 years ago

Sort: -m long option --merge was missing, so I added it, and took the opportunity to make another commit refactoring some of the code, making it simpler.

I see that we rework the whole error treatment of sort, I'll probably be able to fix it soon, looks simple but will need a lot of restructuring to make it more suitable for the other flags to be implemented, cause in the way things are now, those flags won't fit in the program functions.

GrayJack commented 4 years ago

bors r+

GrayJack commented 4 years ago

bors r+

bors[bot] commented 4 years ago

Build succeeded: