summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFedja Beader <fedja@protonmail.ch>2025-03-22 01:51:48 +0100
committerFedja Beader <fedja@protonmail.ch>2025-04-06 01:44:21 +0200
commit252f6f159bd96279a5111b328e5dca349e1832a1 (patch)
tree53b3906de0507978ca9aa97d532887edaac582b9
parentc9d7b8b46cb59b0505efbb163b15d8c503cec3fb (diff)
downloadmanaplus-252f6f159bd96279a5111b328e5dca349e1832a1.tar.gz
manaplus-252f6f159bd96279a5111b328e5dca349e1832a1.tar.bz2
manaplus-252f6f159bd96279a5111b328e5dca349e1832a1.tar.xz
manaplus-252f6f159bd96279a5111b328e5dca349e1832a1.zip
findNearestLivingBeing: remove redundant branch
as without filtering, priority and closestPriority are always the same **** mana/plus!145
-rw-r--r--src/actormanager.cpp22
1 files changed, 6 insertions, 16 deletions
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;