Review fix: Avoid redundant function calls.

Hirotaka Azuma · Hirotaka Azuma · commit d7ccbb3d4255 · 2022-08-27T14:19:01.000Z
diff --git a/tokio-postgres/src/config.rs b/tokio-postgres/src/config.rs
@@ -378,7 +378,7 @@ impl Config {
 
     /// Gets the time interval between TCP keepalive probes.
     pub fn get_keepalives_interval(&self) -> Option<Duration> {
-        self.keepalive_config.interval.as_ref().copied()
+        self.keepalive_config.interval
     }
 
     /// Sets the maximum number of TCP keepalive probes that will be sent before dropping a connection.
@@ -391,7 +391,7 @@ impl Config {
 
     /// Gets the maximum number of TCP keepalive probes that will be sent before dropping a connection.
     pub fn get_keepalives_retries(&self) -> Option<u32> {
-        self.keepalive_config.retries.as_ref().copied()
+        self.keepalive_config.retries
     }
 
     /// Sets the requirements of the session.

Original file line number	Diff line number	Diff line change
`@@ -378,7 +378,7 @@ impl Config {`
`378`	`378`
`379`	`379`	`/// Gets the time interval between TCP keepalive probes.`
`380`	`380`	`pub fn get_keepalives_interval(&self) -> Option<Duration> {`
`381`		`- self.keepalive_config.interval.as_ref().copied()`
	`381`	`+ self.keepalive_config.interval`
`382`	`382`	`}`
`383`	`383`
`384`	`384`	`/// Sets the maximum number of TCP keepalive probes that will be sent before dropping a connection.`
`@@ -391,7 +391,7 @@ impl Config {`
`391`	`391`
`392`	`392`	`/// Gets the maximum number of TCP keepalive probes that will be sent before dropping a connection.`
`393`	`393`	`pub fn get_keepalives_retries(&self) -> Option<u32> {`
`394`		`- self.keepalive_config.retries.as_ref().copied()`
	`394`	`+ self.keepalive_config.retries`
`395`	`395`	`}`
`396`	`396`
`397`	`397`	`/// Sets the requirements of the session.`