Dennis/footswitch-rs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Replaced -v/--value by -i/--input

Browse Source 
Users could make the mistake and set -V, which is valid in every scrope,
in stead of -v. This could result in for the user unexplainable
behaviour.
This commit is contained in:
Dennis Potter 2018-07-26 18:44:28 +02:00
parent acf5729180
commit 1e7907358f
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/main.rs
View File

@ -43,9 +43,9 @@ enum Command {
#[structopt(short = "c", long = "command")] #[structopt(short = "c", long = "command")]
command: Vec<String>, command: Vec<String>,
/// Value to apply /// Input values to apply
#[structopt(short = "v", long = "value")] #[structopt(short = "i", long = "input")]
value: Vec<String>, input: Vec<String>,
}, },
#[structopt(name = "read")] #[structopt(name = "read")]
@ -63,10 +63,11 @@ enum Command {
fn main() { fn main() {
let mut pedals = pedal_operations::Pedals::new(); let mut pedals = pedal_operations::Pedals::new();
let opt = Opt::from_args();
welcome("footswitch-rs, Dennis Potter <dennis@dennispotter.eu>"); welcome("footswitch-rs, Dennis Potter <dennis@dennispotter.eu>");
check_sudo(); check_sudo();
let opt = Opt::from_args();
// All options that don't need the device to be open // All options that don't need the device to be open
// Print all keys and exit application // Print all keys and exit application
@ -104,9 +105,9 @@ fn main() {
// All options that need the device to be open // All options that need the device to be open
match opt.cmd { match opt.cmd {
Some(Command::Write {pedal: ped_list, command: cmd_list, value: val_list}) => { Some(Command::Write {pedal: ped_list, command: cmd_list, input: val_list}) => {
if ped_list.len() != cmd_list.len() && ped_list.len() != val_list.len() { if ped_list.len() != cmd_list.len() && ped_list.len() != val_list.len() {
error!("You must define as much pedals as you define commands and as you define values!"); error!("You must define as much pedals as you define commands and as you define input values!");
} }
for (i, cmd) in cmd_list.iter().enumerate() { for (i, cmd) in cmd_list.iter().enumerate() {
@ -130,7 +131,7 @@ fn main() {
pedals.write_pedals(& dev); pedals.write_pedals(& dev);
info!("Succesfully wrote everything to footpedal!"); info!("Succesfully wrote everything to footpedal!");
info!("The current state of the device is shown below.\n"); info!("The current state of the device is shown below.");
// Show user current state of pedal // Show user current state of pedal
pedals.read_pedals(&dev, vec![0,1,2]); pedals.read_pedals(&dev, vec![0,1,2]);