miniscript: rename fn dissat_sat to sat_dissat

The name of the function no longer implies its return value (which is a
struct rather than a tuple that needs to be carefully ordered). So change
the name to be consistent with the name of the module.

Author: apoelstra

src/miniscript/satisfy/mod.rs

@@ -1026,7 +1026,7 @@ impl<Pk: MiniscriptKey + ToPublicKey> Satisfaction<Placeholder<Pk>> {
         Ctx: ScriptContext,
         P: AssetProvider<Pk>,
     {
-        Self::dissat_sat(node, provider, false, root_has_sig, leaf_hash).sat
+        Self::sat_dissat(node, provider, false, root_has_sig, leaf_hash).sat
     }
 
     pub(crate) fn build_template_mall<P, Ctx>(
@@ -1039,7 +1039,7 @@ impl<Pk: MiniscriptKey + ToPublicKey> Satisfaction<Placeholder<Pk>> {
         Ctx: ScriptContext,
         P: AssetProvider<Pk>,
     {
-        Self::dissat_sat(node, provider, true, root_has_sig, leaf_hash).sat
+        Self::sat_dissat(node, provider, true, root_has_sig, leaf_hash).sat
     }
 
     // produce a non-malleable satisafaction for thesh frag

src/miniscript/satisfy/sat_dissat.rs

@@ -310,7 +310,7 @@ impl<Pk: MiniscriptKey + ToPublicKey> Satisfaction<Placeholder<Pk>> {
     /// If the `malleable` flag is set to true, more efficient satisfactions may be found,
     /// but which a 3rd party may be able to replace with less efficient versions. (This
     /// flag does not affect dissatisfactions.)
-    pub(super) fn dissat_sat<Ctx, Sat>(
+    pub(super) fn sat_dissat<Ctx, Sat>(
         node: &Miniscript<Pk, Ctx>,
         stfr: &Sat,
         malleable: bool,