rust: replace most "operation" structs with functions The hg-core crate has a partially-formed concept of "operation", represented as structs with constructors and a `run` method. Each struct?s contructor takes different parameters, and each `run` has a different return type. Constructors typically don?t do much more than store parameters for `run` to access them. There was a comment about adding an `Operation` trait when the language supports expressing something so general, but it?s hard to imagine how operations with such different APIs could be used in a generic context. This commit starts removing the concept of "operation", since those are pretty much just functions. Differential Revision: https://phab.mercurial-scm.org/D9595

     1

use crate::commands::Command;

     2

use crate::error::CommandError;

     3

use crate::ui::Ui;

     4

use format_bytes::format_bytes;

     5

use hg::operations::find_root;

     6

use hg::utils::files::get_bytes_from_path;

     7

     8

pub const HELP_TEXT: &str = "

     9

Print the root directory of the current repository.

    10

    11

Returns 0 on success.

    12

";

    13

    14

pub struct RootCommand {}

    15

    16

impl RootCommand {

    17

    pub fn new() -> Self {

    18

        RootCommand {}

    19

    }

    20

}

    21

    22

impl Command for RootCommand {

    23

    fn run(&self, ui: &Ui) -> Result<(), CommandError> {

    24

        let path_buf = find_root()?;

    25

    26

        let bytes = get_bytes_from_path(path_buf);

    27

    28

        ui.write_stdout(&format_bytes!(b"{}\n", bytes.as_slice()))?;

    29

    30

        Ok(())

    31

    }

    32

}