Remove TODO

coverage_map/src/lib.rs

@@ -37,9 +37,6 @@ impl CoverageMapBuilder {
     /// A submap only contains the hexes that exist in the provided `coverage_objs` arguments. This
     /// allows for one to determine the potential ranking of new coverage objects without having
     /// to clone the entire CoverageMapBuilder.
-    // TODO(map): Should this return a `CoverageMap` instead? I don't really see the purpose of
-    // having this return a `CoverageMapBuilder` since it will probably always be converted instantly
-    // to a `CoverageMap`.
     pub fn submap(&self, coverage_objs: Vec<CoverageObject>) -> Self {
         // A different way to implement this function would be to insert all of the coverage_objs into
         // the submap, and then reconstruct the coverage objs from only the relevant hexes and then