I'm not sure why it's different here

[Cupnfish]

pub fn get_mut(&mut self, handle: &Handle<T>) -> Option<&mut T> {
        self.events.send(AssetEvent::Modified { handle: *handle });
        self.assets.get_mut(&handle)
    }

    pub fn get_id_mut(&mut self, id: HandleId) -> Option<&mut T> {
        self.assets.get_mut(&Handle::from_id(id))
    }

I'm working on Bevy's code,Then I found this place here, I think it should be rewritten like this

    pub fn get_id_mut(&mut self, id: HandleId) -> Option<&mut T> {
        self.get_mut(&Handle::from_id(id))
    }

[DJMcNab]

Canonical links:

bevy/crates/bevy_asset/src/assets.rs

Lines 75 to 78 in d00ce1c

	pub fn get_mut(&mut self, handle: &Handle<T>) -> Option<&mut T> {
	self.events.send(AssetEvent::Modified { handle: *handle });
	self.assets.get_mut(&handle)
	}

bevy/crates/bevy_asset/src/assets.rs

Lines 67 to 69 in d00ce1c

	pub fn get_id_mut(&mut self, id: HandleId) -> Option<&mut T> {
	self.assets.get_mut(&Handle::from_id(id))
	}

And I think I agree it should be changed to call the former.