chiark / gitweb /
mformat 2: Insist that builtin libraries have an explicit version
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Wed, 11 May 2022 20:44:42 +0000 (21:44 +0100)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Wed, 11 May 2022 23:19:17 +0000 (00:19 +0100)
We don't want to be setting the bad example of a missing `format`, and
of course we don't want to be using format=1 either.

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

index 83d39f92b4b5c50b10d7fdabfb5535d7aa1dd71f..d853b3b2cdd18b1cd5a53191a827f920020f572d 100644 (file)
@@ -910,7 +910,7 @@ impl LibrarySource for BuiltinLibrary<'_> {
 
   #[throws(materials_format::VersionError)]
   fn default_materials_format(&self) -> materials_format::Version {
-    1.try_into().unwrap() // TODO
+    throw!(MFVE::Other("builtin libraries must have explicit version now!"));
   }
 }