subst: Provide Dollars::enabled

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/src/shapelib.rs b/src/shapelib.rs
index 159e5951fc8c4e42994f80f7b8b9b32f437bf3a4..ef46c23b0be7a1b2f3647db1506688b6ae8a743e 100644 (file)
--- a/src/shapelib.rs
+++ b/src/shapelib.rs
@@ -1057,12 +1057,7 @@ impl<'s> Substituting<'s> {
     }.s.into()
   }
 
-  fn do_dollars(&self) -> bool {
-    match self.dollars {
-      Dollars::Filename => false,
-      Dollars::Text => self.mformat >= 2,
-    }
-  }
+  fn do_dollars(&self) -> bool { self.dollars.enabled(self.mformat) }
 
   #[throws(SubstError)]
   /// Expand, but do not do final unescaping
@@ -1082,6 +1077,15 @@ impl<'s> Substituting<'s> {
   }
 }
 
+impl Dollars {
+  fn enabled(self, mformat: materials_format::Version) -> bool {
+    match self {
+      Dollars::Filename => false,
+      Dollars::Text => mformat >= 2,
+    }
+  }
+}
+
 #[throws(SubstError)]
 fn subst_general_precisely<'i>(input: &Substituting<'i>,
                                needle: & str,