Switch 'match' to 'if let' in main.rs::after
This lets us drop a level of indentation, and remove the clunky match arm for Ok(()) => () :)
This commit is contained in:
parent
4eb78cd75a
commit
aecdc7a273
43
src/main.rs
43
src/main.rs
|
@ -91,30 +91,27 @@ async fn dispatch_error(ctx: &Context, msg: &Message, error: DispatchError) {
|
|||
#[hook]
|
||||
async fn after(ctx: &Context, msg: &Message, command_name: &str, command_result: CommandResult) {
|
||||
// prints error in chat
|
||||
match command_result {
|
||||
Ok(()) => (),
|
||||
Err(why) => {
|
||||
let _ = msg
|
||||
.channel_id
|
||||
.send_message(&ctx.http, |m| {
|
||||
m.embed(|e| {
|
||||
e.title(format!("Error in **{}**", command_name))
|
||||
.description(&why.to_string())
|
||||
/*.thumbnail("https://i.imgur.com/VzOEz2E.png") oh no */
|
||||
.colour(0xff6961)
|
||||
})
|
||||
if let Err(why) = command_result {
|
||||
let _ = msg
|
||||
.channel_id
|
||||
.send_message(&ctx.http, |m| {
|
||||
m.embed(|e| {
|
||||
e.title(format!("Error in **{}**", command_name))
|
||||
.description(&why.to_string())
|
||||
/*.thumbnail("https://i.imgur.com/VzOEz2E.png") oh no */
|
||||
.colour(0xff6961)
|
||||
})
|
||||
.await;
|
||||
// prints error in console
|
||||
eprintln!(
|
||||
"{}",
|
||||
format!(
|
||||
"Error in {}: {}",
|
||||
command_name.purple(),
|
||||
&why.to_string().red().bold()
|
||||
)
|
||||
);
|
||||
}
|
||||
})
|
||||
.await;
|
||||
// prints error in console
|
||||
eprintln!(
|
||||
"{}",
|
||||
format!(
|
||||
"Error in {}: {}",
|
||||
command_name.purple(),
|
||||
&why.to_string().red().bold()
|
||||
)
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue