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

Make animation::animate const #99

Open
3 tasks
johannesugb opened this issue Apr 7, 2021 · 0 comments
Open
3 tasks

Make animation::animate const #99

johannesugb opened this issue Apr 7, 2021 · 0 comments
Labels
C++ C++-centric task enhancement New feature or request

Comments

@johannesugb
Copy link
Member

Would be cool if there was a const-version of animation::animate because then one instance of animate could be used from different threads in parallel. The current version assigns to mGlobalTransform and therefore, cannot be const. An alternative would be to duplcate instances of aimation and use the duplicated instances from different threads.

Definition of done:

  • Either a version of animation::animate that is const has been added, ...
  • ... or it has been made sure that animation can be properly copied for the purpose of parallel animations.
  • The code is well documented and the Contribution Guidelines have been followed.
@johannesugb johannesugb added enhancement New feature or request C++ C++-centric task labels Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ C++-centric task enhancement New feature or request
Development

No branches or pull requests

1 participant