Skip to content
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

remove common prefix before comparing numerical filenames #1868

Closed
wants to merge 1 commit into from

Conversation

azuline
Copy link
Contributor

@azuline azuline commented Apr 16, 2024

Currently, nnn sorts files like this:

image

After code change:

image

@N-R-K
Copy link
Collaborator

N-R-K commented Apr 16, 2024

Isn't this what version sort already does? E.g press t and then v (or use -Tv cli option).

@azuline
Copy link
Contributor Author

azuline commented Apr 16, 2024

just tried it; yep! going to close this now

i think what i did there also doesn't create a total ordering, so it wouldn't be a good change anyways without some more analysis.

@azuline azuline closed this Apr 16, 2024
@azuline azuline deleted the prefix-strcmp branch April 16, 2024 04:41
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants