Code optimizations / refactor (#12704)

Co-authored-by: SmallJoker <SmallJoker@users.noreply.github.com>
Co-authored-by: sfan5 <sfan5@live.de>

1 files changed, 3 insertions, 7 deletions

diff --git a/src/nodetimer.h b/src/nodetimer.h
index 40e54ccc5..d952b1cfb 100644
--- a/src/nodetimer.h
+++ b/src/nodetimer.h
@@ -93,15 +93,11 @@ public:
 		}
 	}
 	// Undefined behaviour if there already is a timer
-	void insert(NodeTimer timer) {
+	void insert(const NodeTimer &timer) {
 		v3s16 p = timer.position;
 		double trigger_time = m_time + (double)(timer.timeout - timer.elapsed);
-		std::multimap<double, NodeTimer>::iterator it =
-			m_timers.insert(std::pair<double, NodeTimer>(
-				trigger_time, timer
-			));
-		m_iterators.insert(
-			std::pair<v3s16, std::multimap<double, NodeTimer>::iterator>(p, it));
+		std::multimap<double, NodeTimer>::iterator it = m_timers.emplace(trigger_time, timer);
+		m_iterators.emplace(p, it);
 		if (m_next_trigger_time == -1. || trigger_time < m_next_trigger_time)
 			m_next_trigger_time = trigger_time;
 	}