Mutator Not working

[toast9]

I'm trying to set a bool in one class to false through another class. But no matter what I do the bool stays true. Why????

//this class sets the bool in iter->second->getAnim().setRunning(!mIsPaused)
//mIsPaused is changed from false to true if the space bar is hit.
 void UnitManager::draw()
{
	if (!(mUnitMap.empty()))
	{
		std::map<int, Unit*>::iterator iter;

		//go through all entries in map and delete
		for (iter = mUnitMap.begin(); iter != mUnitMap.end(); ++iter)
		{
			iter->second->getAnim().setRunning(!mIsPaused);
			iter->second->draw();
		}
	}
}

//this function will run depending on that bool

void Animation::update()
{
	//mIsRunning = false;
	if (mIsRunning)
	{
		if (mTimer.getElapsedTime() >= 1600 / mFps)
		{
			mCurrentFrame++;
			//reset the timer
			mTimer.start();
		}

		if (mCurrentFrame >= mNumOfFrames)
			mCurrentFrame = 0;
	}
}

[LB]

Show the source to void setRunning(bool)

[toast9]

public:
	void setRunning(bool running);

private:
	bool mIsRunning;

[toast9]

void Animation::setRunning(bool running)
{
	mIsRunning = running;
}

[cire]

That's the declaration and a variable. It is not the source (the code for the body) of the function.

The return type from getAnim() would also be relevant.

[toast9]

and one more

Animation Unit::getAnim()
{
	return mAnim;
}

[toast9]

It really can't get any simpler. I must be missing something little. I've just been staring at it for too long, and it's evading me. I need a new set of eyes to look at it. Any help is greatly appreciated.

[cire]

There's your problem. Unit::getAnim() returns a copy of an Animation. You're making changes to that copy and not the original.

[toast9]

but mAnim is a member variable of the Unit class. It's returning that unit's mAnim. If not, how would I return the actual mAmin?

[toast9]

Ah got it. Thanks cire. I added the & in the function definition and source.