Merge pull request #110400 from Ryan-000/Optimize-Animation_track_update_hash

Repiteo · Repiteo · commit b72937587a66 · 2025-10-27T10:01:45.000-05:00
Optimize Animation::_track_update_hash
diff --git a/scene/resources/animation.cpp b/scene/resources/animation.cpp
@@ -1062,9 +1062,9 @@ Animation::TrackType Animation::get_cache_type(TrackType p_type) {
 }
 
 void Animation::_track_update_hash(int p_track) {
-	NodePath track_path = tracks[p_track]->path;
-	TrackType track_cache_type = get_cache_type(tracks[p_track]->type);
-	tracks[p_track]->thash = StringName(String(track_path.get_concatenated_names()) + String(track_path.get_concatenated_subnames()) + itos(track_cache_type)).hash();
+	const NodePath &track_path = tracks[p_track]->path;
+	const TrackType track_cache_type = get_cache_type(tracks[p_track]->type);
+	tracks[p_track]->thash = HashMapHasherDefault::hash(Pair<const NodePath &, TrackType>(track_path, track_cache_type));
 }
 
 Animation::TypeHash Animation::track_get_type_hash(int p_track) const {

Original file line number	Diff line number	Diff line change
`@@ -1062,9 +1062,9 @@ Animation::TrackType Animation::get_cache_type(TrackType p_type) {`
`1062`	`1062`	`}`
`1063`	`1063`
`1064`	`1064`	`void Animation::_track_update_hash(int p_track) {`
`1065`		`- NodePath track_path = tracks[p_track]->path;`
`1066`		`- TrackType track_cache_type = get_cache_type(tracks[p_track]->type);`
`1067`		`- tracks[p_track]->thash = StringName(String(track_path.get_concatenated_names()) + String(track_path.get_concatenated_subnames()) + itos(track_cache_type)).hash();`
	`1065`	`+ const NodePath &track_path = tracks[p_track]->path;`
	`1066`	`+ const TrackType track_cache_type = get_cache_type(tracks[p_track]->type);`
	`1067`	`+ tracks[p_track]->thash = HashMapHasherDefault::hash(Pair<const NodePath &, TrackType>(track_path, track_cache_type));`
`1068`	`1068`	`}`
`1069`	`1069`
`1070`	`1070`	`Animation::TypeHash Animation::track_get_type_hash(int p_track) const {`