feat: add list and check incompatible args

src/cli.rs

@@ -6,6 +6,9 @@ use std::path::PathBuf;
 #[clap(author, version, about, long_about = None)]
 pub struct Cli {
     #[clap(value_parser)]
-    pub map_file: PathBuf,
+    pub map_file: Option<PathBuf>,
+    /// List devices and exit
+    #[clap(long, short, action)]
+    pub list: bool,
 }
 

src/config/device.rs

@@ -55,17 +55,14 @@ impl TryFrom<DeviceConfigSerializer> for DeviceConfig {
     fn try_from(v: DeviceConfigSerializer) -> Result<Self, Self::Error> {
         Ok(DeviceConfig {
             identifier: {
-                if v.name.is_some() {
+                match (v.name, v.regex, v.addr) {
-                    Identifier::Name(v.name.unwrap())
+                    (Some(_), Some(_), _      ) => return Err(Error::IncompatibleArgs("name","regex")),
-                }
+                    (Some(_), None   , Some(_)) => return Err(Error::IncompatibleArgs("name","addr")),
-                else if v.regex.is_some() {
+                    (None   , Some(_), Some(_)) => return Err(Error::IncompatibleArgs("regex","addr")),
-                    Identifier::Regex(regex::Regex::new(&v.regex.unwrap())?)
+                    (Some(n), None,    None   ) => Identifier::Name(n),
-                }
+                    (None,    Some(r), None   ) => Identifier::Regex(regex::Regex::new(&r)?),
-                else if v.addr.is_some() {
+                    (None,    None   , Some(a)) => Identifier::Addr(a),
-                    Identifier::Addr(v.addr.unwrap())
+                    (None,    None,    None   ) => Identifier::All,
-                }
-                else {
-                    Identifier::All
                 }
             },
             max_connections: v.max_connections,

src/error.rs

@@ -31,6 +31,10 @@ pub enum Error {
     RemapTooBig(f64),
     #[error("remap value is too low. Minimum value is {}", i64::MIN)]
     RemapTooLow(f64),
+    #[error("'{0}' and '{1}' are incompatible")]
+    IncompatibleArgs(&'static str, &'static str),
+    #[error("no map file was provided")]
+    NoArgument,
     #[error("pipe error")]
     Pipe,
     #[error("unknown error")]

src/main.rs

@@ -21,13 +21,29 @@ use cli::Cli;
 
 fn main() {
     let c = Cli::parse();
+    if c.list {
+        err_handle(run::list_devices());
+        return;
+    }
+    let map_file = err_handle(
+        c.map_file.ok_or(Error::NoArgument)
+    );
     loop {
-        match run_file(&c.map_file) {
+        err_handle(
-            Ok(_) => (),
+            run_file(&map_file)
-            Err(err) => {
+        );
-                eprintln!("Error: {}", err);
+    }
-                std::process::exit(1);
+}
-            }
+
+fn err_handle<T,E>(r: Result<T, E>) -> T
+where
+    E: std::fmt::Display
+{
+    match r {
+        Ok(v) => v,
+        Err(err) => {
+            eprintln!("Error: {}", err);
+            std::process::exit(1);
         }
     }
 }

src/midi/mod.rs

@@ -45,6 +45,15 @@ pub enum MidiPortHandler {
     ALSA(MidiPort<alsa::DeviceAddr>),
 }
 
+impl std::fmt::Display for MidiPortHandler {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        match self {
+            MidiPortHandler::ALSA(p) => write!(f, "{} {}:{}", p.name, p.addr.client, p.addr.port),
+            _ => todo!(),
+        }
+    }
+}
+
 pub enum MidiHandler {
     ALSA(MidiInputAlsa),
 }

src/run.rs

@@ -24,6 +24,15 @@ pub fn cross_shell(cmd: &str) -> Vec<String> {
     ).collect()
 }
 
+pub fn list_devices() -> Result<(), Error> {
+    let input = MidiHandler::new("rmidimap")?;
+    let ports = input.ports()?;
+    for p in ports {
+        println!("{}", p);
+    }
+    Ok(())
+}
+
 pub fn run_config(conf: &Config) -> Result<(), Error> {
     let cfevmap: Vec<DeviceRunItem> = conf.devices.iter().map(|x|
         (x, EventMap::from(x),