Merge pull request #138 from subspace/todo-cleanups

TODO cleanups
autonomys · Mar 1, 2024 · 10d6d51 · 10d6d51
2 parents 3ed8bbd + d440ed4
commit 10d6d51
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/src/backend.rs b/src/backend.rs
@@ -113,8 +113,7 @@ impl PieceGetterWrapper {
     }
 }
 
-// TODO: Derive `Debug` after https://github.com/subspace/subspace/pull/2573
-#[derive(Clone)]
+#[derive(Debug, Clone)]
 struct WeakPieceGetterWrapper {
     farmer_piece_getter: WeakFarmerPieceGetter<
         SegmentCommitmentPieceValidator<MaybeNodeRpcClient>,

diff --git a/src/backend/farmer.rs b/src/backend/farmer.rs
@@ -389,9 +389,7 @@ pub(super) async fn create_farmer(farmer_options: FarmerOptions) -> anyhow::Resu
         .map(|single_disk_farm| async {
             InitialFarmState {
                 total_sectors_count: single_disk_farm.total_sectors_count(),
-                // TODO: Should be `SectorSize` from the beginning
-                plotted_sectors_count: single_disk_farm.plotted_sectors_count().await
-                    as SectorIndex,
+                plotted_sectors_count: single_disk_farm.plotted_sectors_count().await,
             }
         })
         .collect::<FuturesOrdered<_>>()