minor: fix clippy for 1.52.0 (#335)

Author: patrickfreed

src/client/options/mod.rs

@@ -1103,7 +1103,7 @@ impl ClientOptionsParser {
                     None
                 } else {
                     let port_string_without_colon = &port[1..];
-                    let p = u16::from_str_radix(port_string_without_colon, 10).map_err(|_| {
+                    let p: u16 = port_string_without_colon.parse().map_err(|_| {
                         ErrorKind::ArgumentError {
                             message: format!(
                                 "invalid port specified in connection string: {}",
@@ -1348,7 +1348,7 @@ impl ClientOptionsParser {
 
         macro_rules! get_duration {
             ($value:expr, $option:expr) => {
-                match u64::from_str_radix($value, 10) {
+                match $value.parse::<u64>() {
                     Ok(i) => i,
                     _ => {
                         return Err(ErrorKind::ArgumentError {
@@ -1365,7 +1365,7 @@ impl ClientOptionsParser {
 
         macro_rules! get_u32 {
             ($value:expr, $option:expr) => {
-                match u32::from_str_radix(value, 10) {
+                match value.parse::<u32>() {
                     Ok(u) => u,
                     Err(_) => {
                         return Err(ErrorKind::ArgumentError {
@@ -1382,7 +1382,7 @@ impl ClientOptionsParser {
 
         macro_rules! get_i32 {
             ($value:expr, $option:expr) => {
-                match i32::from_str_radix(value, 10) {
+                match value.parse::<i32>() {
                     Ok(u) => u,
                     Err(_) => {
                         return Err(ErrorKind::ArgumentError {
@@ -1654,7 +1654,7 @@ impl ClientOptionsParser {
             "w" => {
                 let mut write_concern = self.write_concern.get_or_insert_with(Default::default);
 
-                match i32::from_str_radix(value, 10) {
+                match value.parse::<i32>() {
                     Ok(w) => match u32::try_from(w) {
                         Ok(uw) => write_concern.w = Some(Acknowledgment::from(uw)),
                         Err(_) => {

src/cmap/establish/handshake/mod.rs

@@ -224,8 +224,8 @@ impl Handshaker {
         });
 
         Ok(HandshakeResult {
-            first_round,
             is_master_reply,
+            first_round,
         })
     }
 }

src/cmap/mod.rs

@@ -86,9 +86,9 @@ impl ConnectionPool {
             address,
             manager,
             connection_requester,
+            generation_subscriber,
             wait_queue_timeout,
             event_handler,
-            generation_subscriber,
         }
     }
 

src/coll/options.rs

@@ -412,8 +412,8 @@ pub struct FindOneAndUpdateOptions {
 /// Specifies the options to a [`Collection::aggregate`](../struct.Collection.html#method.aggregate)
 /// operation.
 #[skip_serializing_none]
-#[serde(rename_all = "camelCase")]
 #[derive(Clone, Debug, Default, Deserialize, TypedBuilder, Serialize)]
+#[serde(rename_all = "camelCase")]
 #[builder(field_defaults(default, setter(strip_option)))]
 #[non_exhaustive]
 pub struct AggregateOptions {

src/test/util/matchable.rs

@@ -17,7 +17,7 @@ pub trait Matchable: Sized + 'static {
         if expected.is_placeholder() {
             return true;
         }
-        if let Some(expected) = Any::downcast_ref::<Self>(expected) {
+        if let Some(expected) = <dyn Any>::downcast_ref::<Self>(expected) {
             self.content_matches(expected)
         } else {
             false