Skip to content

Commit 2d45df5

Browse files
euankLinuxMercedes
authored andcommitted
Remove some unneeded indentation
The change to 'each_incoming' means we can use one fewer match.
1 parent ab959ad commit 2d45df5

1 file changed

Lines changed: 36 additions & 40 deletions

File tree

src/main.rs

Lines changed: 36 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -237,55 +237,51 @@ fn main() {
237237
fn run_irc(server: IrcClient, raw: bool, quiet: bool, sjoin: chan::Sender<Action>) {
238238
server.identify().unwrap_or_else(|err| sjoin.send(From::from(err)));
239239

240-
let res = server.for_each_incoming(|message| {
241-
match message {
242-
msg => {
243-
if raw {
244-
print!("{}", msg);
245-
stdout().flush().unwrap_or_else(|err| sjoin.send(From::from(err)));
246-
}
247-
match msg.command {
248-
Command::JOIN(ref _channel, ref _a, ref _b) => sjoin.send(Action::Join),
249-
Command::PRIVMSG(ref target, ref what_was_said) => {
250-
if !raw && !quiet {
251-
println!("{}->{}: {}",
240+
let res = server.for_each_incoming(|msg| {
241+
if raw {
242+
print!("{}", msg);
243+
stdout().flush().unwrap_or_else(|err| sjoin.send(From::from(err)));
244+
}
245+
match msg.command {
246+
Command::JOIN(ref _channel, ref _a, ref _b) => sjoin.send(Action::Join),
247+
Command::PRIVMSG(ref target, ref what_was_said) => {
248+
if !raw && !quiet {
249+
println!("{}->{}: {}",
252250
msg.source_nickname().unwrap_or("* "),
253251
target,
254252
what_was_said);
255-
stdout().flush().unwrap_or_else(|err| sjoin.send(From::from(err)));
256-
}
257-
}
258-
Command::NOTICE(ref target, ref what_was_said) => {
259-
if !raw && !quiet {
260-
println!("{}->{}: {}",
253+
stdout().flush().unwrap_or_else(|err| sjoin.send(From::from(err)));
254+
}
255+
}
256+
Command::NOTICE(ref target, ref what_was_said) => {
257+
if !raw && !quiet {
258+
println!("{}->{}: {}",
261259
msg.source_nickname().unwrap_or("* "),
262260
target,
263261
what_was_said);
264-
stdout().flush().unwrap_or_else(|err| sjoin.send(From::from(err)));
265-
}
266-
}
267-
Command::QUIT(ref _quitmessage) => sjoin.send(Action::Quit),
268-
Command::Response(ref response, ref command, ref err) => {
269-
// Handle un-joinable channels
270-
let the_problem = command.get(1).map_or("", |s| &*s);
271-
let errmsg = the_problem.to_string() + ": " +
272-
&err.clone().unwrap_or("".to_string());
273-
274-
match *response {
275-
Response::ERR_CHANNELISFULL => sjoin.send(Action::JoinFail(errmsg)),
276-
Response::ERR_INVITEONLYCHAN => sjoin.send(Action::JoinFail(errmsg)),
277-
Response::ERR_BANNEDFROMCHAN => sjoin.send(Action::JoinFail(errmsg)),
278-
Response::ERR_BADCHANNELKEY => sjoin.send(Action::JoinFail(errmsg)),
279-
Response::ERR_NOSUCHCHANNEL => sjoin.send(Action::JoinFail(errmsg)),
280-
281-
Response::RPL_ENDOFMOTD => sjoin.send(Action::Connect),
282-
Response::ERR_NOMOTD => sjoin.send(Action::Connect),
283-
_ => (),
284-
}
285-
}
262+
stdout().flush().unwrap_or_else(|err| sjoin.send(From::from(err)));
263+
}
264+
}
265+
Command::QUIT(ref _quitmessage) => sjoin.send(Action::Quit),
266+
Command::Response(ref response, ref command, ref err) => {
267+
// Handle un-joinable channels
268+
let the_problem = command.get(1).map_or("", |s| &*s);
269+
let errmsg = the_problem.to_string() + ": " +
270+
&err.clone().unwrap_or("".to_string());
271+
272+
match *response {
273+
Response::ERR_CHANNELISFULL => sjoin.send(Action::JoinFail(errmsg)),
274+
Response::ERR_INVITEONLYCHAN => sjoin.send(Action::JoinFail(errmsg)),
275+
Response::ERR_BANNEDFROMCHAN => sjoin.send(Action::JoinFail(errmsg)),
276+
Response::ERR_BADCHANNELKEY => sjoin.send(Action::JoinFail(errmsg)),
277+
Response::ERR_NOSUCHCHANNEL => sjoin.send(Action::JoinFail(errmsg)),
278+
279+
Response::RPL_ENDOFMOTD => sjoin.send(Action::Connect),
280+
Response::ERR_NOMOTD => sjoin.send(Action::Connect),
286281
_ => (),
287282
}
288283
}
284+
_ => (),
289285
}
290286
});
291287
if let Err(err) = res {

0 commit comments

Comments
 (0)