Fix some clippy lints
This commit is contained in:
parent
2ccd8a9c55
commit
c7fd5f8ff1
6 changed files with 11 additions and 13 deletions
|
@ -41,8 +41,7 @@ pub async fn parse_site_logo(
|
|||
let elem = elem_ref.value();
|
||||
let ancestors = elem_ref
|
||||
.ancestors()
|
||||
.map(ElementRef::wrap)
|
||||
.flatten()
|
||||
.filter_map(ElementRef::wrap)
|
||||
.map(|elem_ref| elem_ref.value())
|
||||
.collect::<Vec<_>>();
|
||||
|
||||
|
@ -119,7 +118,7 @@ pub async fn parse_site_logo(
|
|||
let href = if elem.name() == "svg" {
|
||||
Some(Url::parse(&encode_svg(&elem_ref.html())).unwrap())
|
||||
} else {
|
||||
elem.attr("src").and_then(|href| url.join(&href).ok())
|
||||
elem.attr("src").and_then(|href| url.join(href).ok())
|
||||
};
|
||||
|
||||
if let Some(href) = &href {
|
||||
|
|
|
@ -38,7 +38,7 @@ impl TryFrom<&str> for IconSizes {
|
|||
type Error = Box<dyn Error>;
|
||||
|
||||
fn try_from(sizes_str: &str) -> Result<Self, Self::Error> {
|
||||
let size_strs = sizes_str.split(" ");
|
||||
let size_strs = sizes_str.split(' ');
|
||||
|
||||
let mut sizes = Vec::new();
|
||||
for size in size_strs {
|
||||
|
@ -85,7 +85,7 @@ impl IntoIterator for IconSizes {
|
|||
|
||||
impl Ord for IconSizes {
|
||||
fn cmp(&self, other: &Self) -> Ordering {
|
||||
self.largest().cmp(&other.largest())
|
||||
self.largest().cmp(other.largest())
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -28,10 +28,10 @@ pub async fn get_jpeg_size<R: AsyncRead + Unpin>(
|
|||
let page = marker[1];
|
||||
|
||||
// Check for valid SOFn markers. C4, C8, and CC aren't dimension markers.
|
||||
if (page >= 0xC0 && page <= 0xC3)
|
||||
|| (page >= 0xC5 && page <= 0xC7)
|
||||
|| (page >= 0xC9 && page <= 0xCB)
|
||||
|| (page >= 0xCD && page <= 0xCF)
|
||||
if (0xC0..=0xC3).contains(&page)
|
||||
|| (0xC5..=0xC7).contains(&page)
|
||||
|| (0xC9..=0xCB).contains(&page)
|
||||
|| (0xCD..=0xCF).contains(&page)
|
||||
{
|
||||
// Only get outside image size
|
||||
if depth == 0 {
|
||||
|
|
|
@ -71,7 +71,7 @@ impl<'de> Deserialize<'de> for IconSize {
|
|||
{
|
||||
let value: String = Deserialize::deserialize(deserializer)?;
|
||||
|
||||
let mut split = value.split("x");
|
||||
let mut split = value.split('x');
|
||||
let width = split
|
||||
.next()
|
||||
.ok_or(de::Error::custom("expected width"))?
|
||||
|
|
|
@ -129,8 +129,7 @@ impl SiteIcons {
|
|||
}
|
||||
.boxed_local(),
|
||||
async {
|
||||
let manifests =
|
||||
join_all(manifest_urls.map(|url| SiteIcons::load_manifest(url))).await;
|
||||
let manifests = join_all(manifest_urls.map(SiteIcons::load_manifest)).await;
|
||||
|
||||
LoadedKind::DefaultManifest(
|
||||
manifests
|
||||
|
|
|
@ -49,6 +49,6 @@ async fn load_manifest_cached(url: Url) -> Result<Vec<Icon>, String> {
|
|||
}))
|
||||
.await
|
||||
.into_iter()
|
||||
.filter_map(|icon| icon)
|
||||
.flatten()
|
||||
.collect())
|
||||
}
|
||||
|
|
Reference in a new issue