diff options
author | Andrei Karas <akaras@inbox.ru> | 2013-02-23 00:00:43 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2013-02-23 00:00:43 +0300 |
commit | c4e562fffe134b86a35800dbe906afcf761879bc (patch) | |
tree | f6884b020d29745a2e9c859372c058d67476b6bc | |
parent | a3ecadac89eafbacd0c6b3764d5977bcac242c57 (diff) | |
download | mv-c4e562fffe134b86a35800dbe906afcf761879bc.tar.gz mv-c4e562fffe134b86a35800dbe906afcf761879bc.tar.bz2 mv-c4e562fffe134b86a35800dbe906afcf761879bc.tar.xz mv-c4e562fffe134b86a35800dbe906afcf761879bc.zip |
add const and some minor changes in compoundsprite class.
-rw-r--r-- | src/compoundsprite.cpp | 18 | ||||
-rw-r--r-- | src/compoundsprite.h | 3 |
2 files changed, 4 insertions, 17 deletions
diff --git a/src/compoundsprite.cpp b/src/compoundsprite.cpp index 01e144bf4..7f7e554a9 100644 --- a/src/compoundsprite.cpp +++ b/src/compoundsprite.cpp @@ -66,23 +66,18 @@ CompoundSprite::CompoundSprite() : CompoundSprite::~CompoundSprite() { clear(); - -// delete mImage; mImage = nullptr; -// delete mAlphaImage; mAlphaImage = nullptr; } bool CompoundSprite::reset() { bool ret = false; - FOR_EACH (SpriteIterator, it, mSprites) { if (*it) ret |= (*it)->reset(); } - mNeedsRedraw |= ret; return ret; } @@ -90,13 +85,11 @@ bool CompoundSprite::reset() bool CompoundSprite::play(const std::string &action) { bool ret = false; - FOR_EACH (SpriteIterator, it, mSprites) { if (*it) ret |= (*it)->play(action); } - mNeedsRedraw |= ret; return ret; } @@ -104,13 +97,11 @@ bool CompoundSprite::play(const std::string &action) bool CompoundSprite::update(const int time) { bool ret = false; - FOR_EACH (SpriteIterator, it, mSprites) { if (*it) ret |= (*it)->update(time); } - mNeedsRedraw |= ret; return ret; } @@ -132,9 +123,8 @@ bool CompoundSprite::draw(Graphics *const graphics, else if (mAlpha && mAlphaImage) { mAlphaImage->setAlpha(mAlpha); - return graphics->drawImage(mAlphaImage, - posX + mOffsetX, posY + mOffsetY); + posX + mOffsetX, posY + mOffsetY); } else { @@ -206,13 +196,11 @@ const Image *CompoundSprite::getImage() const bool CompoundSprite::setSpriteDirection(const SpriteDirection direction) { bool ret = false; - FOR_EACH (SpriteIterator, it, mSprites) { if (*it) ret |= (*it)->setSpriteDirection(direction); } - mNeedsRedraw |= ret; return ret; } @@ -232,7 +220,6 @@ unsigned int CompoundSprite::getCurrentFrame() const if (*it) return (*it)->getCurrentFrame(); } - return 0; } @@ -243,7 +230,6 @@ unsigned int CompoundSprite::getFrameCount() const if (*it) return (*it)->getFrameCount(); } - return 0; } @@ -304,7 +290,7 @@ void CompoundSprite::ensureSize(size_t layerCount) /** * Returns the curent frame in the current animation of the given layer. */ -unsigned int CompoundSprite::getCurrentFrame(unsigned int layer) +unsigned int CompoundSprite::getCurrentFrame(unsigned int layer) const { if (layer >= mSprites.size()) return 0; diff --git a/src/compoundsprite.h b/src/compoundsprite.h index 928c6b06f..777b09363 100644 --- a/src/compoundsprite.h +++ b/src/compoundsprite.h @@ -116,7 +116,8 @@ public: /** * Returns the curent frame in the current animation of the given layer. */ - virtual unsigned int getCurrentFrame(unsigned int layer) A_WARN_UNUSED; + virtual unsigned int getCurrentFrame(unsigned int layer) + const A_WARN_UNUSED; /** * Returns the frame count in the current animation of the given layer. |