changeset 45380:227281e76c22

rhg: Do not return error when when we really mean ok in commands Before when a command was successfull `Err(CommandErrorKind::Ok.into())` was returned which is an oxymoron. Using `Ok(())` when everything is ok seems more appropriate. Differential Revision: https://phab.mercurial-scm.org/D8864
author Antoine Cezar <antoine.cezar@octobus.net>
date Tue, 21 Jul 2020 10:39:30 +0200
parents 0f5286ccf82c
children 47997afadf08
files rust/rhg/src/commands/root.rs rust/rhg/src/error.rs
diffstat 2 files changed, 1 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/rust/rhg/src/commands/root.rs	Sat Aug 08 12:52:39 2020 -0700
+++ b/rust/rhg/src/commands/root.rs	Tue Jul 21 10:39:30 2020 +0200
@@ -29,7 +29,7 @@
         // TODO use formating macro
         self.ui.write_stdout(&[bytes.as_slice(), b"\n"].concat())?;
 
-        Err(CommandErrorKind::Ok.into())
+        Ok(())
     }
 
     fn display_error(&self, error: FindRootError) -> Result<(), CommandError> {
--- a/rust/rhg/src/error.rs	Sat Aug 08 12:52:39 2020 -0700
+++ b/rust/rhg/src/error.rs	Tue Jul 21 10:39:30 2020 +0200
@@ -5,8 +5,6 @@
 /// The kind of command error
 #[derive(Debug, PartialEq)]
 pub enum CommandErrorKind {
-    /// The command finished without error
-    Ok,
     /// The root of the repository cannot be found
     RootNotFound,
     /// The current directory cannot be found
@@ -20,7 +18,6 @@
 impl CommandErrorKind {
     pub fn get_exit_code(&self) -> exitcode::ExitCode {
         match self {
-            CommandErrorKind::Ok => exitcode::OK,
             CommandErrorKind::RootNotFound => exitcode::ABORT,
             CommandErrorKind::CurrentDirNotFound => exitcode::ABORT,
             CommandErrorKind::StdoutError => exitcode::ABORT,