refactor: add --reverse flag to sort

helix-term/src/commands/typed.rs

@@ -2123,35 +2123,33 @@ fn language(
 
 fn sort(
     cx: &mut compositor::Context,
-    args: Args,
+    mut args: Args,
     flags: &[Flag],
     event: PromptEvent,
 ) -> anyhow::Result<()> {
     if event != PromptEvent::Validate {
         return Ok(());
     }
 
-    sort_impl(cx, args, flags, false)
-}
+    if let Some(flag) = args.next() {
+        let flag = flag.trim_start_matches("--").trim_start_matches('-');
 
-fn sort_reverse(
-    cx: &mut compositor::Context,
-    args: Args,
-    flags: &[Flag],
-    event: PromptEvent,
-) -> anyhow::Result<()> {
-    if event != PromptEvent::Validate {
-        return Ok(());
+        if flags
+            .iter()
+            .any(|x| x.long == flag || x.short == Some(flag))
+        {
+            sort_impl(cx, true);
+        } else {
+            bail!("There is no flag `--{flag}` on sort");
+        }
+    } else {
+        sort_impl(cx, false);
     }
-    sort_impl(cx, args, flags, true)
+
+    Ok(())
 }
 
-fn sort_impl(
-    cx: &mut compositor::Context,
-    _args: Args,
-    _flags: &[Flag],
-    reverse: bool,
-) -> anyhow::Result<()> {
+fn sort_impl(cx: &mut compositor::Context, reverse: bool) {
     let scrolloff = cx.editor.config().scrolloff;
     let (view, doc) = current!(cx.editor);
     let text = doc.text().slice(..);
@@ -2179,8 +2177,6 @@ fn sort_impl(
     doc.apply(&transaction, view.id);
     doc.append_changes_to_history(view);
     view.ensure_cursor_in_view(doc, scrolloff);
-
-    Ok(())
 }
 
 fn reflow(
@@ -3206,14 +3202,6 @@ pub const TYPABLE_COMMAND_LIST: &[TypableCommand] = &[
         fun: sort,
         signature: CommandSignature::none(),
     },
-    TypableCommand {
-        name: "rsort",
-        aliases: &[],
-        flags: &[],
-        doc: "Sort ranges in selection in reverse order.",
-        fun: sort_reverse,
-        signature: CommandSignature::none(),
-    },
     TypableCommand {
         name: "reflow",
         aliases: &[],