Skip to content

Conversation

@ivanpauno
Copy link
Contributor

Depends on #195.
Ports osrf#3

@ivanpauno ivanpauno self-assigned this Nov 16, 2021
Base automatically changed from ivanpauno/port-osrf-#2 to main November 16, 2021 18:10
Signed-off-by: Chris Lalancette <[email protected]>
This should make the code much easier to read.

Signed-off-by: Chris Lalancette <[email protected]>
This makes it a lot nicer to use.

Signed-off-by: Chris Lalancette <[email protected]>
1.  Check return value of malloc.
2.  Get rid of unnecessary cast.

Signed-off-by: Chris Lalancette <[email protected]>
@ivanpauno ivanpauno force-pushed the ivanpauno/port-osrf-#3 branch from 321408c to 9fbb260 Compare November 16, 2021 18:12
@jacobperron
Copy link
Contributor

FYI, @esteve
I don't recall if you wanted a ping on each PR as we upstream changes from our fork.

Signed-off-by: Ivan Santiago Paunovic <[email protected]>
@ivanpauno ivanpauno merged commit ae5edaa into main Nov 16, 2021
@jacobperron jacobperron deleted the ivanpauno/port-osrf-#3 branch November 16, 2021 21:37
@esteve
Copy link
Member

esteve commented Nov 17, 2021

FYI, @esteve
I don't recall if you wanted a ping on each PR as we upstream changes from our fork.

Thank you guys for your hard work and for keeping me in the loop. I won't have time to review every PR, but it's nice to see the progress of your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants