Contiguous novel atoms are always better than changing depth of novel lists

CHANGELOG.md

@@ -12,6 +12,9 @@ Difftastic will now prefer matching up comments that are similar
 Contiguous syntax logic now considers close delimiter positions, so
 `[ \n ];` now treats the `;` atom as contiguous.
 
+Fixed an issue where diffs would prefer prefer a low depth change on a
+delimiter over a delimiter that gave contiguous changes.
+
 ### Display
 
 Display width calculations are now based on the longest line visible

src/dijkstra.rs

@@ -156,14 +156,20 @@ impl Edge {
                 if *contiguous {
                     300
                 } else {
-                    301
+                    // This needs to be more than 40 greater than the
+                    // contiguous case. Otherwise, we end up choosing
+                    // a badly positioned unchanged elimiter just
+                    // because it has a better depth difference.
+                    //
+                    // TODO: write a test for this case.
+                    350
                 }
             }
             NovelDelimiterLHS { contiguous } | NovelDelimiterRHS { contiguous } => {
                 if *contiguous {
                     300
                 } else {
-                    301
+                    350
                 }
             }