-
Notifications
You must be signed in to change notification settings - Fork 305
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
[all] Add DeprecatedData and RemovedData #3934
[all] Add DeprecatedData and RemovedData #3934
Conversation
Sofa/framework/Core/src/sofa/core/objectmodel/DeprecatedData.cpp
Outdated
Show resolved
Hide resolved
[ci-build][with-tests] |
[ci-build][with-all-tests] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, except some added indentations for the license citation in some files, and for some comments as well 🤐
...s/MultiThreading/src/MultiThreading/component/animationloop/AnimationLoopParallelScheduler.h
Outdated
Show resolved
Hide resolved
...s/MultiThreading/src/MultiThreading/component/animationloop/AnimationLoopParallelScheduler.h
Outdated
Show resolved
Hide resolved
2 tests are considered as failing since the message of RemovedData is thrown as an |
Co-authored-by: Alex Bilger <alxbilger@users.noreply.github.com>
Co-authored-by: Alex Bilger <alxbilger@users.noreply.github.com>
Signed-off-by: Damien Marchal <damien.marchal@univ-lille1.fr>
[ci-build][with-all-tests] |
* WIP * Generalize the use of RemoveData instead of overriding parse() * FIXUP * FIXUP: Add SOFA_CORE_API * WIp * Update Sofa/framework/Core/src/sofa/core/objectmodel/RemovedData.h Co-authored-by: Alex Bilger <alxbilger@users.noreply.github.com> * Update Sofa/framework/Core/src/sofa/core/objectmodel/DeprecatedData.h Co-authored-by: Alex Bilger <alxbilger@users.noreply.github.com> * Fix broken indentation * Replace the deprecated "numberThread" with nbThreads Signed-off-by: Damien Marchal <damien.marchal@univ-lille1.fr> --------- Signed-off-by: Damien Marchal <damien.marchal@univ-lille1.fr> Co-authored-by: Alex Bilger <alxbilger@users.noreply.github.com>
To make it more simple to manage deprecation and more specifically to not override the parse() method for that purpose.
By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).
Reviewers will merge this pull-request only if