Implement Display trait for BlockStatsFields

Author: Gabriel Comte

client/src/client.rs

@@ -471,7 +471,7 @@ pub trait RpcApi: Sized {
         height: u64,
         fields: &[json::BlockStatsFields],
     ) -> Result<json::GetBlockStatsResultPartial> {
-        let fields: Vec<&str> = fields.iter().map(|field| field.get_rpc_keyword()).collect();
+        let fields: Vec<String> = fields.iter().map(|field| field.to_string()).collect();
 
         self.call("getblockstats", &[height.into(), fields.into()])
     }

json/src/lib.rs

@@ -31,6 +31,7 @@ use bitcoin::util::{bip158, bip32};
 use bitcoin::{Address, Amount, PrivateKey, PublicKey, Script, SignedAmount, Transaction};
 use serde::de::Error as SerdeError;
 use serde::{Deserialize, Serialize};
+use std::fmt;
 
 //TODO(stevenroose) consider using a Time type
 
@@ -435,7 +436,7 @@ pub enum BlockStatsFields {
 }
 
 impl BlockStatsFields {
-    pub fn get_rpc_keyword(&self) -> &str {
+    fn get_rpc_keyword(&self) -> &str {
         match *self {
             BlockStatsFields::AverageFee => "avgfee",
             BlockStatsFields::AverageFeeRate => "avgfeerate",
@@ -470,6 +471,12 @@ impl BlockStatsFields {
     }
 }
 
+impl fmt::Display for BlockStatsFields {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        write!(f, "{}", self.get_rpc_keyword())
+    }
+}
+
 #[derive(Clone, PartialEq, Debug, Deserialize, Serialize)]
 #[serde(rename_all = "camelCase")]
 pub struct GetMiningInfoResult {
@@ -1156,7 +1163,6 @@ impl<'de> serde::Deserialize<'de> for ImportMultiRescanSince {
         D: serde::Deserializer<'de>,
     {
         use serde::de;
-        use std::fmt;
         struct Visitor;
         impl<'de> de::Visitor<'de> for Visitor {
             type Value = ImportMultiRescanSince;