Use is_*() instead of if let x(_), fixes redundant_pattern_matching
This commit is contained in:
parent
b4aa23ecf5
commit
d496eaf3bb
|
@ -1218,7 +1218,7 @@ impl TextureManager {
|
||||||
}
|
}
|
||||||
|
|
||||||
fn get_texture(&self, name: &str) -> Option<Texture> {
|
fn get_texture(&self, name: &str) -> Option<Texture> {
|
||||||
if let Some(_) = name.find(':') {
|
if name.find(':').is_some() {
|
||||||
self.textures.get(name).cloned()
|
self.textures.get(name).cloned()
|
||||||
} else {
|
} else {
|
||||||
self.textures.get(&format!("minecraft:{}", name)).cloned()
|
self.textures.get(&format!("minecraft:{}", name)).cloned()
|
||||||
|
|
|
@ -131,7 +131,7 @@ impl Manager {
|
||||||
// (if it was started)
|
// (if it was started)
|
||||||
let mut done = false;
|
let mut done = false;
|
||||||
if let Some(ref recv) = self.vanilla_chan {
|
if let Some(ref recv) = self.vanilla_chan {
|
||||||
if let Ok(_) = recv.try_recv() {
|
if recv.try_recv().is_ok() {
|
||||||
done = true;
|
done = true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -141,7 +141,7 @@ impl Manager {
|
||||||
}
|
}
|
||||||
let mut done = false;
|
let mut done = false;
|
||||||
if let Some(ref recv) = self.vanilla_assets_chan {
|
if let Some(ref recv) = self.vanilla_assets_chan {
|
||||||
if let Ok(_) = recv.try_recv() {
|
if recv.try_recv().is_ok() {
|
||||||
done = true;
|
done = true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -266,7 +266,7 @@ impl Server {
|
||||||
thread::spawn(move || loop {
|
thread::spawn(move || loop {
|
||||||
let pck = read.read_packet();
|
let pck = read.read_packet();
|
||||||
let was_error = pck.is_err();
|
let was_error = pck.is_err();
|
||||||
if let Err(_) = tx.send(pck) {
|
if tx.send(pck).is_err() {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
if was_error {
|
if was_error {
|
||||||
|
|
Loading…
Reference in New Issue