fix(ls): disambiguate size sorting appropiately

Alonely0 · Alonely0 · commit 912c8816e336 · 2026-05-01T16:37:45.000+02:00
diff --git a/src/uu/ls/src/ls.rs b/src/uu/ls/src/ls.rs
@@ -1480,7 +1480,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
`@@ -1480,7 +1480,12 @@ fn sort_entries(entries: &mut [PathData], config: &Config) {`
`1480`	`1480`	`)`
`1481`	`1481`	`}),`
`1482`	`1482`	`Sort::Size => {`
`1483`		`- entries.sort_unstable_by_key(\|k\| Reverse(k.metadata().map_or(0, Metadata::len)));`
	`1483`	`+ entries.sort_unstable_by(\|a, b\| {`
	`1484`	`+ b.metadata()`
	`1485`	`+ .map_or(0, Metadata::len)`
	`1486`	`+ .cmp(&a.metadata().map_or(0, Metadata::len))`
	`1487`	`+ .then(a.file_name().cmp(b.file_name()))`
	`1488`	`+ });`
`1484`	`1489`	`}`
`1485`	`1490`	`// The default sort in GNU ls is case insensitive`
`1486`	`1491`	`Sort::Name => entries.sort_unstable_by(\|a, b\| a.display_name().cmp(b.display_name())),`