Skip to content

list: add list_append_ref#1527

Closed
cspiel1 wants to merge 4 commits intobaresip:mainfrom
cspiel1:list_append_ref
Closed

list: add list_append_ref#1527
cspiel1 wants to merge 4 commits intobaresip:mainfrom
cspiel1:list_append_ref

Conversation

@cspiel1
Copy link
Collaborator

@cspiel1 cspiel1 commented Feb 18, 2026

This allow to append an element to multiple lists

@sreimers
Copy link
Member

Can you add some basic test code?

@cspiel1
Copy link
Collaborator Author

cspiel1 commented Feb 18, 2026

For now we don't need to return the le in lep. Function list_tail() could be used if it is needed. I though if we once add similar functions list_insert_xxx_ref() it would make sense to have the pointer to the new le.

@cspiel1
Copy link
Collaborator Author

cspiel1 commented Feb 19, 2026

I have one improvement. The flag bool ref could be added to the struct list.

@cspiel1
Copy link
Collaborator Author

cspiel1 commented Feb 19, 2026

We came to the conclusion that it is not worth to make list complicated in order to

@cspiel1 cspiel1 closed this Feb 19, 2026
@cspiel1 cspiel1 deleted the list_append_ref branch February 19, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments