chiark / gitweb /
shapelib refactoring: Rename find_load
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Mon, 22 Mar 2021 00:56:23 +0000 (00:56 +0000)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Mon, 22 Mar 2021 00:58:17 +0000 (00:58 +0000)
The old name was very confusing.  Now there are fewer functions called
`load`.

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

index 92b123b89e90f4a2f8f647f9bf9a4904eb305c7b..44c2d95148763f4209186181965607687879a549 100644 (file)
@@ -91,7 +91,7 @@ fn preview(items: Vec<ItemForOutput>) {
   let mut pieces: Vec<Prep> = items.into_iter().map(|it| {
     let spec = ItemSpec { lib: it.0, item: it.1.itemname };
     (||{
-      let (p, _occultable) = spec.clone().load().context("load")?;
+      let (p, _occultable) = spec.clone().find_load().context("load")?;
       // todo show occulted version too
       let mut uos = vec![];
       p.add_ui_operations(&mut uos, &GameState::dummy(), &GPiece::dummy())
index c801462189d16aaf52d085cdb65837a3a5adcd00..178d755d9fddcce62b2dbf27fcb06aa774479b4a 100644 (file)
@@ -291,7 +291,7 @@ impl From<ItemSpecLoaded> for PieceSpecLoaded {
 
 impl ItemSpec {
   #[throws(SpecError)]
-  pub fn load(&self) -> ItemSpecLoaded {
+  pub fn find_load(&self) -> ItemSpecLoaded {
     let lib = libs_lookup(&self.lib)?;
     let idata = lib.items.get(&self.item)
       .ok_or(SpE::LibraryItemNotFound(self.item.clone()))?;
@@ -404,7 +404,7 @@ impl PieceSpec for ItemSpec {
   #[throws(SpecError)]
   fn load(&self, _: usize, _: &mut GPiece, _ir: &InstanceRef)
           -> PieceSpecLoaded {
-    self.load()?.into()
+    self.find_load()?.into()
   }
 }
 
@@ -421,7 +421,7 @@ impl PieceSpec for MultiSpec {
     )?;
     let item = format!("{}{}{}", &self.prefix, item, &self.suffix);
     let lib = self.lib.clone();
-    ItemSpec { lib, item }.load()?.into()
+    ItemSpec { lib, item }.find_load()?.into()
   }
 }