Update comment to remark that Drop is no longer implemented.

Author: rklaehn

src/store/fs.rs

@@ -221,9 +221,6 @@ impl entity_manager::Params for EmParams {
         state: entity_manager::ActiveEntityState<Self>,
         cause: entity_manager::ShutdownCause,
     ) {
-        // this isn't strictly necessary. Drop will run anyway as soon as the
-        // state is reset to it's default value. Doing it here means that we
-        // have exact control over where it happens.
         if let Some(mut handle) = state.state.0.lock().await.take() {
             trace!("shutting down hash: {}, cause: {cause:?}", state.id);
             handle.persist(&state);

src/store/fs/bao_file.rs

@@ -502,6 +502,10 @@ impl BaoFileStorage {
 }
 
 /// A cheaply cloneable handle to a bao file, including the hash and the configuration.
+///
+/// You must call [Self::persist] to write the bitfield to disk, if you want to persist
+/// the file handle, otherwise the bitfield will not be written to disk and will have
+/// to be reconstructed on next use.
 #[derive(Debug, Clone, derive_more::Deref)]
 pub(crate) struct BaoFileHandle(Arc<watch::Sender<BaoFileStorage>>);