Skip to content

Introduce consistent naming for scheme interface functions #1472

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

Open
lukasdreyer opened this issue Mar 11, 2025 · 1 comment
Open

Introduce consistent naming for scheme interface functions #1472

lukasdreyer opened this issue Mar 11, 2025 · 1 comment
Labels
priority:low Should be solved eventually priority:medium Should be solved within half a year

Comments

@lukasdreyer
Copy link
Collaborator

Feature request

Consistent naming for scheme interface functions

Is your feature request related to a problem? Please describe.
Some functions are called get_, set_to_, construct with similar ideas.

Describe the solution or feature you'd like
From mattermost discussion:
get - if information on the current element is returned but the element is not changed (one const input)
set - if something is changed for the Input element (one input = output)
construct - if a new element is build from information of the input element (one const input, one non-const input = output)

Estimated priority
Which of these is most applicable (remove the others):

"Priority: low" Should be solved eventually

@Davknapp Davknapp added priority:medium Should be solved within half a year workload:low Would take half a day or less labels Mar 24, 2025
@lukasdreyer
Copy link
Collaborator Author

Answer from @holke from mattermost:

get - if information on the current element is returned but the element is not changed (one const input)
set - if something is changed for the Input element (one input = output)
construct - if a new element is build from information of the input element (one const input, one non-const input = output)

@ole-alb ole-alb moved this from In-Box to ToDo in t8code's Issue Landing page Apr 8, 2025
@ole-alb ole-alb added priority:low Should be solved eventually and removed workload:low Would take half a day or less labels Apr 8, 2025
@t8ddy t8ddy moved this from ToDo to In-Box in t8code's Issue Landing page Apr 8, 2025
@Davknapp Davknapp moved this from In-Box to ToDo in t8code's Issue Landing page Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low Should be solved eventually priority:medium Should be solved within half a year
Projects
Status: ToDo
Development

No branches or pull requests

3 participants