From 84f8167fd1d54a5ebf924d6d4a43bd24cfe40dd2 Mon Sep 17 00:00:00 2001
From: Nathan Vegdahl <cessen@cessen.com>
Date: Mon, 26 Jul 2021 23:09:58 -0700
Subject: [PATCH] Use `match` for branching on the `Direction` enum in more
 places.

---
 helix-core/src/movement.rs | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/helix-core/src/movement.rs b/helix-core/src/movement.rs
index 4af82a1de..38e14594c 100644
--- a/helix-core/src/movement.rs
+++ b/helix-core/src/movement.rs
@@ -34,10 +34,9 @@ pub fn move_horizontally(
     let pos = range.cursor(slice);
 
     // Compute the new position.
-    let new_pos = if dir == Direction::Backward {
-        nth_prev_grapheme_boundary(slice, pos, count)
-    } else {
-        nth_next_grapheme_boundary(slice, pos, count)
+    let new_pos = match dir {
+        Direction::Forward => nth_next_grapheme_boundary(slice, pos, count),
+        Direction::Backward => nth_prev_grapheme_boundary(slice, pos, count),
     };
 
     // Compute the final new range.
@@ -59,10 +58,9 @@ pub fn move_vertically(
 
     // Compute the new position.
     let (new_pos, new_row) = {
-        let new_row = if dir == Direction::Backward {
-            row.saturating_sub(count)
-        } else {
-            (row + count).min(slice.len_lines().saturating_sub(1))
+        let new_row = match dir {
+            Direction::Forward => (row + count).min(slice.len_lines().saturating_sub(1)),
+            Direction::Backward => row.saturating_sub(count),
         };
         let new_col = col.max(horiz as usize);
         (