nit: quick comment and code cleanuip

Signed-off-by: Carson Farmer <[email protected]>

Author: carsonfarmer

fendermint/actors/objectstore/src/actor.rs

@@ -20,8 +20,8 @@ pub struct Actor;
 
 impl Actor {
     fn constructor(rt: &impl Runtime) -> Result<(), ActorError> {
-        // FIXME: (sander) We're setting this up to be a subnet-wide actor for a single repo.
-        // FIXME: (sander) In the future, this could be deployed dynamically for multi repo subnets.
+        // FIXME:(sander) We're setting this up to be a subnet-wide actor for a single repo.
+        // FIXME:(sander) In the future, this could be deployed dynamically for multi repo subnets.
         rt.validate_immediate_caller_is(std::iter::once(&SYSTEM_ACTOR_ADDR))?;
 
         let state = State::new(rt.store()).map_err(|e| {
@@ -35,7 +35,7 @@ impl Actor {
     }
 
     fn append_object(rt: &impl Runtime, params: PutObjectParams) -> Result<(), ActorError> {
-        // FIXME: (@carsonfarmer) We'll want to validate the caller is the owner of the repo.
+        // FIXME:(carsonfarmer) We'll want to validate the caller is the owner of the repo.
         rt.validate_immediate_caller_accept_any()?;
 
         rt.transaction(|st: &mut State, rt| {
@@ -49,7 +49,7 @@ impl Actor {
     }
 
     fn put_object(rt: &impl Runtime, params: PutObjectParams) -> Result<(), ActorError> {
-        // FIXME: (@carsonfarmer) We'll want to validate the caller is the owner of the repo.
+        // FIXME:(carsonfarmer) We'll want to validate the caller is the owner of the repo.
         rt.validate_immediate_caller_accept_any()?;
 
         rt.transaction(|st: &mut State, rt| {
@@ -63,7 +63,7 @@ impl Actor {
     }
 
     fn delete_object(rt: &impl Runtime, params: DeleteObjectParams) -> Result<(), ActorError> {
-        // FIXME: (@carsonfarmer) We'll want to validate the caller is the owner of the repo.
+        // FIXME:(carsonfarmer) We'll want to validate the caller is the owner of the repo.
         rt.validate_immediate_caller_accept_any()?;
 
         rt.transaction(|st: &mut State, rt| {

fendermint/actors/objectstore/src/shared.rs

@@ -18,7 +18,7 @@ pub struct State {
     pub root: Cid,
 }
 
-// TODO: (@carsonfarmer) We'll likely want to define the metadata type that will actually be placed in the Hamt
+// TODO:(carsonfarmer) We'll likely want to define the metadata type that will actually be placed in the Hamt
 
 impl State {
     pub fn new<BS: Blockstore>(store: &BS) -> anyhow::Result<Self> {
@@ -45,7 +45,6 @@ impl State {
 
         let new_content = match hamt.get(&key)? {
             Some(existing) => {
-                // Append the object to the existing object
                 let mut new_content = existing.clone();
                 new_content.extend(content);
                 new_content
@@ -66,15 +65,14 @@ impl State {
     ) -> anyhow::Result<()> {
         let mut hamt = Hamt::<_, Vec<u8>>::load_with_bit_width(&self.root, store, BIT_WIDTH)?;
 
-        // TODO: We could use set_if_absent here to avoid overwriting existing objects.
+        // TODO:(carsonfarmer) We could use set_if_absent here to avoid overwriting existing objects.
         hamt.set(key, content)?;
         self.root = hamt.flush()?;
         Ok(())
     }
 
     pub fn delete<BS: Blockstore>(&mut self, store: &BS, key: &BytesKey) -> anyhow::Result<()> {
         let mut hamt = Hamt::<_, Vec<u8>>::load_with_bit_width(&self.root, store, BIT_WIDTH)?;
-
         hamt.delete(key)?;
         self.root = hamt.flush()?;
         Ok(())
@@ -86,20 +84,17 @@ impl State {
         key: &BytesKey,
     ) -> anyhow::Result<Option<Vec<u8>>> {
         let hamt = Hamt::<_, Vec<u8>>::load_with_bit_width(&self.root, store, BIT_WIDTH)?;
-
         let value = hamt.get(key).map(|v| v.map(|inner| inner.to_owned()))?;
         Ok(value)
     }
 
     pub fn list<BS: Blockstore>(&self, store: &BS) -> anyhow::Result<Vec<Vec<u8>>> {
         let hamt = Hamt::<_, Vec<u8>>::load_with_bit_width(&self.root, store, BIT_WIDTH)?;
-
         let mut keys = Vec::new();
         hamt.for_each(|k, _| {
             keys.push(k.0.to_owned());
             Ok(())
         })?;
-
         Ok(keys)
     }
 }
@@ -130,9 +125,8 @@ pub enum Method {
 
 #[cfg(test)]
 mod tests {
-    use std::str::FromStr;
-
     use super::*;
+    use std::str::FromStr;
 
     #[test]
     fn test_constructor() {