Use sort.SliceStable to ensure stability

Without this, a file that contains two lines that are identical when viewed case-insensitively can end up flapping between omegasort runs.
houseabsolute · Dec 29, 2020 · 0fce38c · 0fce38c
1 parent c394dff
commit 0fce38c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/Changes.md b/Changes.md
@@ -9,6 +9,9 @@
 * Replace file renaming with copying to handle the case where the temp file we
   sort into and the original file are not on the same partition.
 
+* Fix bug where sorting wasn't stable in the presence of two
+  case-insensitively identical lines (and possibly other similar scenarious)>
+
 
 ## 0.0.3 - 2020-12-27
 

diff --git a/main.go b/main.go
@@ -365,7 +365,7 @@ func (o *omegasort) run() error {
 		return nil
 	}
 
-	sort.Slice(lines, sorter)
+	sort.SliceStable(lines, sorter)
 	if *errRef != nil {
 		return *errRef
 	}