merge move_up/move_down into move_line_up/move_line_down

Author: Juhan280

src/core_editor/editor.rs

@@ -64,8 +64,8 @@ impl Editor {
             EditCommand::MoveToPosition { position, select } => {
                 self.move_to_position(*position, *select)
             }
-            EditCommand::MoveLineUp { select } => self.move_up(*select),
-            EditCommand::MoveLineDown { select } => self.move_down(*select),
+            EditCommand::MoveLineUp { select } => self.move_line_up(*select),
+            EditCommand::MoveLineDown { select } => self.move_line_down(*select),
             EditCommand::MoveLeft { select } => self.move_left(*select),
             EditCommand::MoveRight { select } => self.move_right(*select),
             EditCommand::MoveWordLeft { select } => self.move_word_left(*select),
@@ -254,12 +254,14 @@ impl Editor {
         self.line_buffer.set_insertion_point(position)
     }
 
-    pub(crate) fn move_line_up(&mut self) {
+    pub(crate) fn move_line_up(&mut self, select: bool) {
+        self.update_selection_anchor(select);
         self.line_buffer.move_line_up();
         self.update_undo_state(UndoBehavior::MoveCursor);
     }
 
-    pub(crate) fn move_line_down(&mut self) {
+    pub(crate) fn move_line_down(&mut self, select: bool) {
+        self.update_selection_anchor(select);
         self.line_buffer.move_line_down();
         self.update_undo_state(UndoBehavior::MoveCursor);
     }
@@ -621,16 +623,6 @@ impl Editor {
         self.line_buffer.insert_str(string);
     }
 
-    fn move_up(&mut self, select: bool) {
-        self.update_selection_anchor(select);
-        self.line_buffer.move_line_up();
-    }
-
-    fn move_down(&mut self, select: bool) {
-        self.update_selection_anchor(select);
-        self.line_buffer.move_line_down();
-    }
-
     fn move_left(&mut self, select: bool) {
         self.update_selection_anchor(select);
         self.line_buffer.move_left();

src/engine.rs

@@ -1662,7 +1662,7 @@ impl Reedline {
             // If we're at the top, move to previous history
             self.previous_history();
         } else {
-            self.run_edit_commands(&[EditCommand::MoveLineUp { select: false }]);
+            self.editor.move_line_up(false);
         }
     }
 
@@ -1672,7 +1672,7 @@ impl Reedline {
             // If we're at the top, move to previous history
             self.next_history();
         } else {
-            self.run_edit_commands(&[EditCommand::MoveLineDown { select: false }]);
+            self.editor.move_line_down(false);
         }
     }