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

lwm2m_remove_object(...) return removed object #713

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

parmi93
Copy link
Contributor

@parmi93 parmi93 commented Jun 14, 2023

Small change to the lwm2m_remove_object(...) function so that it returns the pointer to the removed object.

@parmi93
Copy link
Contributor Author

parmi93 commented Jun 16, 2023

Fixed the commit message according to the rules.

@parmi93 parmi93 force-pushed the lwm2m_remove_object_return_obj_removed branch from 1bb504d to 99371c7 Compare June 16, 2023 14:04
Small change to the lwm2m_remove_object(...) function so that it returns
the pointer to the removed object.
@parmi93 parmi93 force-pushed the lwm2m_remove_object_return_obj_removed branch from 99371c7 to 014c637 Compare June 16, 2023 14:05
@parmi93
Copy link
Contributor Author

parmi93 commented Jun 16, 2023

Removed the history of commits, in order to be able to resolve errors related to commit messages that did not respect the rules.

@rettichschnidi
Copy link
Contributor

When exactly will this be useful? Would be great to have this information also in the commit message.

@parmi93
Copy link
Contributor Author

parmi93 commented Oct 16, 2023

This parameter is never used inside the library, but in my case it is useful.
I developed an LwM2M client that allows the user to add and remove LwM2M resources on the fly.

Basically the user sends commands from a terminal to add, modify, and delete LwM2M resources, the parameter added through this PR is useful to me during the removal of a LwM2M resource, in this way once the resource has been removed from Wakamma lib I can perform the free of the pointer returned by 'lwm2m_remove_object(...)'

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