Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RAII wrapper for invocations similar to CallbacksGroup #129

Open
TheRealKaup opened this issue Dec 4, 2024 · 0 comments
Open

RAII wrapper for invocations similar to CallbacksGroup #129

TheRealKaup opened this issue Dec 4, 2024 · 0 comments
Labels
bug Not working properly developed `topic/*` awaiting release engine Engine and its components memory Memory management, safety or optimization topic/* Needs a `topic/*` branch to be merged into `develop`

Comments

@TheRealKaup
Copy link
Owner

Users can accidentally forget to call CancelInvocation() on their yet to be called invoked function in their class destructors.

For instance, this is how it's currently done (example from the Button widget):

	~Button()
	{
		if (m_downInvocation != nullptr)
		{
			engine.time.CancelInvocation(m_downInvocation);
		}
	}
@TheRealKaup TheRealKaup added bug Not working properly memory Memory management, safety or optimization labels Dec 4, 2024
@TheRealKaup TheRealKaup changed the title RAII wrapper for invocations similar to CallbacksGroup. RAII wrapper for invocations similar to CallbacksGroup Dec 7, 2024
@TheRealKaup TheRealKaup added this to the v1.1.0: `StateMachine` milestone Dec 16, 2024
@TheRealKaup TheRealKaup added documentation Doxygen reference, markdown files, etc. and removed documentation Doxygen reference, markdown files, etc. labels Dec 18, 2024
TheRealKaup added a commit that referenced this issue Dec 18, 2024
…(all API changes in Doxygen).

Close #129.

Give `Time` its own `engine/time/` directory.
@TheRealKaup TheRealKaup added engine Engine and its components topic/* Needs a `topic/*` branch to be merged into `develop` developed `topic/*` awaiting release labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Not working properly developed `topic/*` awaiting release engine Engine and its components memory Memory management, safety or optimization topic/* Needs a `topic/*` branch to be merged into `develop`
Projects
None yet
Development

No branches or pull requests

1 participant