CAniImageBox Loading Optimized

[Distraught 5688]

Hello guys,

As we know, if you have a lot of images in an ani-image, it can be really slow to load and may makes the game freeze for a moment. It really limits us, because we can't use a longer image sequence as it would have a bad impact on user experience.

I created a sequence of 126 images by concatenating some of our tutorials, so here's a comparison GIF that shows the old (left) and new (right) version.

The main problem is that it loads all the images in a loop when we initialize it.

And they are loaded when we call AppendImage function.

 

We can delay and separate the loading of images in the update cycle to different frames, so the game won't freeze because just one image per frame will be loaded instead of them all at once.

 

How to do this? Step by step tutorial.

This is the hidden content, please

• Sign In
• or
• Sign Up

 

That's all, we're finished!

Good luck!

Edited July 8, 2022 by Distraught

[Ezequiel G. 406]

Nice, thanks for sharing!

[Kafa 147]

Thank you bro, as always good work!

[Dios 1]

Thanks for ur work.

[meneleos 292]

There is a problem with clearimages function.

	void CAniImageBox::ClearImages()
	{
		while (!m_ImageFileNames.empty())
			m_ImageFileNames.pop();

		for_each(m_ImageVector.begin(), m_ImageVector.end(), CGraphicExpandedImageInstance::DeleteExpandedImageInstance);
		m_ImageVector.clear();
	}

fixed.

 

Edited July 20, 2022 by meneleos

[Gurgarath 2427]

On 7/20/2022 at 2:28 PM, meneleos said:

There is a problem with clearimages function.

	void CAniImageBox::ClearImages()
	{
		while (!m_ImageFileNames.empty())
			m_ImageFileNames.pop();

		for_each(m_ImageVector.begin(), m_ImageVector.end(), CGraphicExpandedImageInstance::DeleteExpandedImageInstance);
		m_ImageVector.clear();
	}

fixed.

 

Are you talking about the destructor? Because there is no "ClearImages" function in clean sources

[meneleos 292]

On 7/25/2022 at 10:29 AM, Gurgarath said:

Are you talking about the destructor? Because there is no "ClearImages" function in clean sources

A necessary function for some systems that were added later.

[Distraught 5688]

7 hours ago, meneleos said:

A necessary function for some systems that were added later.

Then it's not a fix, it's in your code only.

[Ulthar 2783]

On 7/8/2022 at 2:12 AM, Distraught said:

Go to CAniImageBox::SetDiffuseColor function and overwrite with 

	void CAniImageBox::SetDiffuseColor(float fR, float fG, float fB, float fA)
	{
		m_SetDiffuseColor = std::bind(&CGraphicExpandedImageInstance::SetDiffuseColor, std::placeholders::_1, fR, fG, fB, fA);
		std::for_each(m_ImageVector.begin(), m_ImageVector.end(), m_SetDiffuseColor);
	}

Instead of this function,

 have this:

	void CImageBox::SetDiffuseColor(float fr, float fg, float fb, float fa)
	{
		if (!m_pImageInstance)
			return;

		m_pImageInstance->SetDiffuseColor(fr, fg, fb, fa);
	}

Is it okey if i overwrite this?

Edited January 7, 2023 by Ulthar

[Distraught 5688]

That’s the function of another class.

[Ulthar 2783]

29 minutes ago, Distraught said:

That’s the function of another class.

Yea i knew, just asked. so i dont have it, then i do nothing, right ? :'D