Merge #481: Make numeric error conversion consistent

294b0e5 Make numeric error conversion consistent (Jamil Lambert, PhD)

Pull request description:

  Most of the repo uses a bare `?` operator in numeric type conversion, but some places use `.map_err(E::Numeric)?`.

  Use the bare `?` operator everywhere for consistency.

ACKs for top commit:
  tcharding:
    ACK 294b0e5

Tree-SHA512: 33aa57acf496915a762f5ef38ae9d987e2ce10c1bf067779c91c275420dc633388660f7f44d2f0e140e433ebcf769bd167fd8d356f08cf694c439b231ffcdbd7

Author: tcharding

types/src/v26/blockchain/into.rs

@@ -15,13 +15,13 @@ impl GetChainStates {
         use GetChainStatesError as E;
 
         Ok(model::GetChainStates {
-            headers: crate::to_u32(self.headers, "headers").map_err(E::Numeric)?,
+            headers: crate::to_u32(self.headers, "headers")?,
             chain_states: self
                 .chain_states
                 .into_iter()
                 .map(|s| {
                     Ok(model::ChainState {
-                        blocks: crate::to_u32(s.blocks, "blocks").map_err(E::Numeric)?,
+                        blocks: crate::to_u32(s.blocks, "blocks")?,
                         best_block_hash: s.best_block_hash.parse().map_err(E::BestBlockHash)?,
                         bits: None,   // v29 and later only.
                         target: None, // v29 and later only.

types/src/v29/blockchain/into.rs

@@ -176,13 +176,13 @@ impl GetChainStates {
         use GetChainStatesError as E;
 
         Ok(model::GetChainStates {
-            headers: crate::to_u32(self.headers, "headers").map_err(E::Numeric)?,
+            headers: crate::to_u32(self.headers, "headers")?,
             chain_states: self
                 .chain_states
                 .into_iter()
                 .map(|s| {
                     Ok(model::ChainState {
-                        blocks: crate::to_u32(s.blocks, "blocks").map_err(E::Numeric)?,
+                        blocks: crate::to_u32(s.blocks, "blocks")?,
                         best_block_hash: s.best_block_hash.parse().map_err(E::BestBlockHash)?,
                         bits: Some(CompactTarget::from_unprefixed_hex(&s.bits).map_err(E::Bits)?),
                         target: Some(