Style tweaks

src/callbacks.rs

@@ -186,9 +186,9 @@ fn handle_body(mut ctx: Context, bytes: &[u8]) -> milter::Result<Status> {
 
     let max = config::get().max_message_size();
     Ok(if client.bytes_written() > max {
-        client.skip_body();
         let id = queue_id(&ctx.api)?;
         verbose!("{}: skipping rest of message larger than {} bytes", id, max);
+        client.skip_body();
         Status::Skip
     } else {
         Status::Continue

src/collections.rs

@@ -47,17 +47,17 @@ where
     }
 }
 
-// Ad-hoc implementation, only does `to_owned` on the lookup key if the key is
-// not yet present.
+// Ad-hoc implementation, only allocates a lookup key if the key is not yet
+// present.
 impl<V> StrVecMap<String, V> {
     pub fn insert_if_absent<K>(&mut self, key: K, value: V) -> Option<V>
     where
-        K: AsRef<str>,
+        K: AsRef<str> + Into<String>,
     {
         if self.contains_key(key.as_ref()) {
             Some(value)
         } else {
-            self.entries.push((key.as_ref().to_owned(), value));
+            self.entries.push((key.into(), value));
             None
         }
     }