Merge #616: Remove unnecessary helper function

927735e Remove unnecessary helper function (Tobin C. Harding)

Pull request description:

  The `_translate_pk` helper function adds nothing to the code. It comes from a day past when the signature of `translate_pk` and `_translate_pk` differed (see commit ef1d69c).

ACKs for top commit:
  apoelstra:
    ACK 927735e

Tree-SHA512: 9d654a040bb26376379aefe5927188137a73cb7fc5d48a75041db2f851ad5970ec642317b83d976bd5e6d37e6b9d68adc63d0c196b1e93186fc9a622b5ef0c62

Author: apoelstra

src/policy/semantic.rs

@@ -127,14 +127,6 @@ impl<Pk: MiniscriptKey> Policy<Pk> {
     /// assert_eq!(real_policy, expected_policy);
     /// ```
     pub fn translate_pk<Q, E, T>(&self, t: &mut T) -> Result<Policy<Q>, E>
-    where
-        T: Translator<Pk, Q, E>,
-        Q: MiniscriptKey,
-    {
-        self._translate_pk(t)
-    }
-
-    fn _translate_pk<Q, E, T>(&self, t: &mut T) -> Result<Policy<Q>, E>
     where
         T: Translator<Pk, Q, E>,
         Q: MiniscriptKey,
@@ -151,7 +143,7 @@ impl<Pk: MiniscriptKey> Policy<Pk> {
             Policy::Older(n) => Ok(Policy::Older(n)),
             Policy::Threshold(k, ref subs) => {
                 let new_subs: Result<Vec<Policy<Q>>, _> =
-                    subs.iter().map(|sub| sub._translate_pk(t)).collect();
+                    subs.iter().map(|sub| sub.translate_pk(t)).collect();
                 new_subs.map(|ok| Policy::Threshold(k, ok))
             }
         }