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.

Sign up for GitHub

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

Jump to bottom

Rename parsec_add2arena #290

Open
abouteiller opened this issue Apr 8, 2021 · 1 comment · May be fixed by #623
Open

Rename parsec_add2arena #290

abouteiller opened this issue Apr 8, 2021 · 1 comment · May be fixed by #623
Assignees
@abouteiller
Labels
API Change Change to the public API, backward incompatible (version major bump) enhancement New feature or request high priority This is an important feature
Milestone
v5.0

Comments

@abouteiller
Copy link
Contributor

Original report by me.

This name does not make sense anymore, we should simplify it to something less weird/internal looking as it is part of the user-facing interface.
@abouteiller abouteiller added enhancement New feature or request and removed proposal labels Feb 17, 2022
@bosilca
Copy link
Contributor

bosilca commented Oct 7, 2022

  • parsec_add2arena: parsec_arena_datatype_set_matrix_tile (or adt instead od arena_datatype). This function should internally call parsec_arena_datatype_construct
  • parsec_del2arena: should be removed, and all calls replaced with parsec_type_free

@abouteiller abouteiller added API Change Change to the public API, backward incompatible (version major bump) and removed PaRSEC 4.0 labels Oct 12, 2022
@abouteiller abouteiller removed this from the v4.0 milestone Jan 25, 2024
@abouteiller abouteiller linked a pull request Apr 24, 2024 that will close this issue
Make coverity happy for 4.0 #623
Draft
@abouteiller abouteiller self-assigned this Apr 24, 2024
@abouteiller abouteiller added this to the v4.0 milestone Apr 24, 2024
@abouteiller abouteiller modified the milestones: v4.0, v5.0 Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abouteiller abouteiller

Labels
API Change Change to the public API, backward incompatible (version major bump) enhancement New feature or request high priority This is an important feature
Projects
None yet
Milestone
v5.0
Development

Successfully merging a pull request may close this issue.

Make coverity happy for 4.0 bosilca/parsec
3 participants
@bosilca @abouteiller @devreal