diff options
author | Fedja Beader <fedja@protonmail.ch> | 2025-03-22 01:51:48 +0100 |
---|---|---|
committer | Fedja Beader <fedja@protonmail.ch> | 2025-04-06 01:44:21 +0200 |
commit | 252f6f159bd96279a5111b328e5dca349e1832a1 (patch) | |
tree | 53b3906de0507978ca9aa97d532887edaac582b9 | |
parent | c9d7b8b46cb59b0505efbb163b15d8c503cec3fb (diff) | |
download | manaplus-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.cpp | 22 |
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; |