Remove or revert `Default` impls from `MaterialStatManifest` and `AbilityMap`
See discussion here !2231 (comment 565137506) and here !2231 (comment 565136395)
It's mentioned that they need Default
to be in the ecs but we can have ecs resouces that don't implement Default
so that is not a reason to have these implement Default
.
MaterialStatManifest
default impl https://gitlab.com/veloren/veloren/-/blob/3497f4fa12623ccbe7768248b322fdcdddeaa4f8/common/src/comp/inventory/item/tool.rs#L181
AbilityMap
default impl https://gitlab.com/veloren/veloren/-/blob/3497f4fa12623ccbe7768248b322fdcdddeaa4f8/common/src/comp/inventory/item/tool.rs#L391
Note: AbilityMap::default
has a todo note on it to revert to the old version which seems to be:
impl Default for AbilityMap {
fn default() -> Self {
let mut map = HashMap::new();
map.insert(ToolKind::Empty, AbilitySet::default());
AbilityMap(map)
}
}