From 252f6f159bd96279a5111b328e5dca349e1832a1 Mon Sep 17 00:00:00 2001 From: Fedja Beader Date: Sat, 22 Mar 2025 01:51:48 +0100 Subject: findNearestLivingBeing: remove redundant branch as without filtering, priority and closestPriority are always the same **** mana/plus!145 --- src/actormanager.cpp | 22 ++++++---------------- 1 file changed, 6 insertions(+), 16 deletions(-) (limited to 'src/actormanager.cpp') diff --git a/src/actormanager.cpp b/src/actormanager.cpp index 62d438c67..737a864a7 100644 --- a/src/actormanager.cpp +++ b/src/actormanager.cpp @@ -1360,23 +1360,13 @@ Being *ActorManager::findNearestLivingBeing(const Being *const aroundBeing, //logger->log("being prio:%3d, dist^2:%3d, name: '%s'", // priority, distSq, being->getName().c_str()); - if (!filtered) + // without filtering, priority and closestPriority are always the same + if ((priority == closestPriority && distSq < closestDistSq) || + (priority < closestPriority && distSq <= maxDistSq)) { - if (distSq < closestDistSq) - { - closestDistSq = distSq; - closestBeing = being; - } - } - else - { - if ((priority < closestPriority && distSq <= maxDistSq) || - (priority == closestPriority && distSq < closestDistSq)) - { - closestDistSq = distSq; - closestBeing = being; - closestPriority = priority; - } + closestDistSq = distSq; + closestBeing = being; + closestPriority = priority; } } return closestBeing; -- cgit v1.2.3-70-g09d2