From 829c8fb2476f763f0d0a1f4ca792073eb953814e Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Tue, 3 Jan 2017 17:02:07 +0300 Subject: Remove useless todo messages. --- src/being/being.h | 3 --- src/dyetool/gui/viewport.h | 4 ---- src/effectmanager.cpp | 1 - src/gui/viewport.h | 4 ---- src/gui/widgets/tabbedarea.h | 2 -- src/gui/widgets/tabs/setup_chat.cpp | 1 - src/resources/map/map.cpp | 4 +--- src/resources/map/map.h | 4 ---- src/resources/sprite/animatedsprite.cpp | 1 - 9 files changed, 1 insertion(+), 23 deletions(-) diff --git a/src/being/being.h b/src/being/being.h index 9e623cbad..42071b0ab 100644 --- a/src/being/being.h +++ b/src/being/being.h @@ -450,14 +450,12 @@ class Being notfinal : public ActorSprite, /** * Sets the attack speed. - * @todo In what unit? */ void setAttackSpeed(const int speed) restrict2 noexcept2 { mAttackSpeed = speed; } /** * Gets the attack speed. - * @todo In what unit? */ int getAttackSpeed() const restrict2 noexcept2 A_WARN_UNUSED { return mAttackSpeed; } @@ -1200,7 +1198,6 @@ class Being notfinal : public ActorSprite, /** * Calculates the offset in the given directions. * If walking in direction 'neg' the value is negated. - * TODO: Used by eAthena only? */ template int getOffset() const restrict2 A_WARN_UNUSED; diff --git a/src/dyetool/gui/viewport.h b/src/dyetool/gui/viewport.h index ffcd17034..847a00dc2 100644 --- a/src/dyetool/gui/viewport.h +++ b/src/dyetool/gui/viewport.h @@ -39,10 +39,6 @@ class MapItem; /** * The viewport on the map. Displays the current map and handles mouse input * and the popup menu. - * - * TODO: This class is planned to be extended to allow floating widgets on top - * of it such as NPC messages, which are positioned using map pixel - * coordinates. */ class Viewport final : public WindowContainer, public MouseListener, diff --git a/src/effectmanager.cpp b/src/effectmanager.cpp index 2d5bb0556..32d441b31 100644 --- a/src/effectmanager.cpp +++ b/src/effectmanager.cpp @@ -204,7 +204,6 @@ bool EffectManager::trigger(const int id, } if (!effect.sfx.empty()) soundManager.playSfx(effect.sfx); - // TODO add sprite effect to position return rValue; } } diff --git a/src/gui/viewport.h b/src/gui/viewport.h index 5af86606c..f7360c42a 100644 --- a/src/gui/viewport.h +++ b/src/gui/viewport.h @@ -46,10 +46,6 @@ class MapItem; /** * The viewport on the map. Displays the current map and handles mouse input * and the popup menu. - * - * TODO: This class is planned to be extended to allow floating widgets on top - * of it such as NPC messages, which are positioned using map pixel - * coordinates. */ class Viewport final : public WindowContainer, public MouseListener, diff --git a/src/gui/widgets/tabbedarea.h b/src/gui/widgets/tabbedarea.h index 16968e614..4e609970d 100644 --- a/src/gui/widgets/tabbedarea.h +++ b/src/gui/widgets/tabbedarea.h @@ -108,8 +108,6 @@ class TabbedArea final : public ActionListener, /** * Return how many tabs have been created. - * - * @todo Remove this method when upgrading to Guichan 0.9.0 */ int getNumberOfTabs() const A_WARN_UNUSED; diff --git a/src/gui/widgets/tabs/setup_chat.cpp b/src/gui/widgets/tabs/setup_chat.cpp index 3f6e8cbb3..c9d0565b9 100644 --- a/src/gui/widgets/tabs/setup_chat.cpp +++ b/src/gui/widgets/tabs/setup_chat.cpp @@ -126,7 +126,6 @@ Setup_Chat::Setup_Chat(const Widget2 *const widget) : "Note: chat logs can take noticeable amount of disk space over time."), "enableChatLog", this, "enableChatLogEvent"); - // TODO: ask 4144 if popup description is correct. // TRANSLATORS: settings option new SetupItemCheckBox(_("Enable debug chat Log"), // TRANSLATORS: settings description diff --git a/src/resources/map/map.cpp b/src/resources/map/map.cpp index 92dfe0c7d..c4a4e7fa8 100644 --- a/src/resources/map/map.cpp +++ b/src/resources/map/map.cpp @@ -963,9 +963,7 @@ Path Map::findPath(const int startX, const int startY, ? basicCost : basicCost2); /* Demote an arbitrary direction to speed pathfinding by - adding a defect (TODO: change depending on the desired - visual effect, e.g. a cross-product defect toward - destination). + adding a defect Important: as long as the total defect along any path is less than the basicCost, the pathfinder will still find one of the shortest paths! */ diff --git a/src/resources/map/map.h b/src/resources/map/map.h index 83d187baf..d9811d8b9 100644 --- a/src/resources/map/map.h +++ b/src/resources/map/map.h @@ -100,10 +100,6 @@ class Map final : public Properties, /** * Draws the map to the given graphics output. This method draws all * layers, actors and overlay effects. - * - * TODO: For efficiency reasons, this method could take into account - * the clipping rectangle set on the Graphics object. However, - * currently the map is always drawn full-screen. */ void draw(Graphics *restrict const graphics, int scrollX, diff --git a/src/resources/sprite/animatedsprite.cpp b/src/resources/sprite/animatedsprite.cpp index ad90a3344..960a5ea00 100644 --- a/src/resources/sprite/animatedsprite.cpp +++ b/src/resources/sprite/animatedsprite.cpp @@ -434,7 +434,6 @@ const void *AnimatedSprite::getHash() const restrict2 bool AnimatedSprite::updateNumber(const unsigned num) restrict2 { - // TODO need store num in delayed object if it exist for future usage if (!mSprite) return false; -- cgit v1.2.3-60-g2f50