clippy fix

Author: YuhanLiin

micropb-gen/src/config.rs

@@ -637,12 +637,11 @@ fn substitute_param<'a>(
     pat: &str,
     t: Option<impl ToString>,
 ) -> Cow<'a, str> {
-    if let Some(t) = t {
-        if typestr.find(pat).is_some() {
+    if let Some(t) = t
+        && typestr.find(pat).is_some() {
             let t = t.to_string();
             return typestr.replace(pat, &t).into();
         }
-    }
     typestr
 }
 

micropb-gen/src/generator/location.rs

@@ -64,7 +64,7 @@ pub(crate) fn add_location_comments(
     };
 
     let path = &location.path;
-    if path.len() % 2 != 0 {
+    if !path.len().is_multiple_of(2) {
         return;
     }
     let segments: Vec<_> = path.chunks(2).map(|chunk| (chunk[0], chunk[1])).collect();

micropb-gen/src/generator/type_spec.rs

@@ -475,13 +475,12 @@ impl<'proto> TypeSpec<'proto> {
     /// generator options). Even though strings and bytes are also length-delimited, we don't need
     /// to cache them because calculating their lengths is trivial.
     pub(crate) fn generate_cache_type(&self, ctx: &Context<'proto>) -> Option<TokenStream> {
-        if let TypeSpec::Message(tname) = self {
-            if ctx.params.cache_extern_types || !ctx.params.extern_paths.contains_key(*tname) {
+        if let TypeSpec::Message(tname) = self
+            && (ctx.params.cache_extern_types || !ctx.params.extern_paths.contains_key(*tname)) {
                 let cache_name = (*tname).to_owned() + "._Cache";
                 let cache_type = ctx.resolve_type_name(&cache_name);
                 return Some(cache_type);
             }
-        }
         None
     }