fix(ls): disambiguate size sorting appropiately

Alonely0 · cakebaker · commit 779cb058cd74 · 2026-05-05T11:22:45.000+02:00
diff --git a/src/uu/ls/src/ls.rs b/src/uu/ls/src/ls.rs
@@ -1481,7 +1481,12 @@ fn sort_entries(entries: &mut [PathData], config: &Config) {
             )
         }),
         Sort::Size => {
-            entries.sort_unstable_by_key(|k| Reverse(k.metadata().map_or(0, Metadata::len)));
+            entries.sort_unstable_by(|a, b| {
+                b.metadata()
+                    .map_or(0, Metadata::len)
+                    .cmp(&a.metadata().map_or(0, Metadata::len))
+                    .then(a.file_name().cmp(b.file_name()))
+            });
         }
         // The default sort in GNU ls is case insensitive
         Sort::Name => entries.sort_unstable_by(|a, b| a.display_name().cmp(b.display_name())),

Original file line number	Diff line number	Diff line change
`@@ -1481,7 +1481,12 @@ fn sort_entries(entries: &mut [PathData], config: &Config) {`
`1481`	`1481`	`)`
`1482`	`1482`	`}),`
`1483`	`1483`	`Sort::Size => {`
`1484`		`- entries.sort_unstable_by_key(\|k\| Reverse(k.metadata().map_or(0, Metadata::len)));`
	`1484`	`+ entries.sort_unstable_by(\|a, b\| {`
	`1485`	`+ b.metadata()`
	`1486`	`+ .map_or(0, Metadata::len)`
	`1487`	`+ .cmp(&a.metadata().map_or(0, Metadata::len))`
	`1488`	`+ .then(a.file_name().cmp(b.file_name()))`
	`1489`	`+ });`
`1485`	`1490`	`}`
`1486`	`1491`	`// The default sort in GNU ls is case insensitive`
`1487`	`1492`	`Sort::Name => entries.sort_unstable_by(\|a, b\| a.display_name().cmp(b.display_name())),`