Review fix: get_keepalives_{interval,retries} returns copied Option.

Author: Hirotaka Azuma

postgres/src/config.rs

@@ -293,7 +293,7 @@ impl Config {
     }
 
     /// Gets the time interval between TCP keepalive probes.
-    pub fn get_keepalives_interval(&self) -> Option<&Duration> {
+    pub fn get_keepalives_interval(&self) -> Option<Duration> {
         self.config.get_keepalives_interval()
     }
 
@@ -306,7 +306,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> {
+    pub fn get_keepalives_retries(&self) -> Option<u32> {
         self.config.get_keepalives_retries()
     }
 

tokio-postgres/src/config.rs

@@ -377,8 +377,8 @@ impl Config {
     }
 
     /// Gets the time interval between TCP keepalive probes.
-    pub fn get_keepalives_interval(&self) -> Option<&Duration> {
-        self.keepalive_config.interval.as_ref()
+    pub fn get_keepalives_interval(&self) -> Option<Duration> {
+        self.keepalive_config.interval.as_ref().copied()
     }
 
     /// Sets the maximum number of TCP keepalive probes that will be sent before dropping a connection.
@@ -390,8 +390,8 @@ 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()
+    pub fn get_keepalives_retries(&self) -> Option<u32> {
+        self.keepalive_config.retries.as_ref().copied()
     }
 
     /// Sets the requirements of the session.