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

Adding C++ API #57

Open
4 tasks
jrsnen opened this issue Jul 14, 2021 · 2 comments
Open
4 tasks

Adding C++ API #57

jrsnen opened this issue Jul 14, 2021 · 2 comments
Assignees
@RamadanAhmed
Labels
feature New feature

Comments

@jrsnen
Copy link
Member

jrsnen commented Jul 14, 2021

Currently, uvgRTP public API is very much C-like. It would be highly beneficial to add separate C++ API alongside the C API so smart pointers can be used.

This would not break the existing API. The C API will be supported at least until 3.0 release of uvgRTP and possibly long after it.

Task lists:

  • Creation of session with smart pointers
  • Creation of media_streams with smart pointers
  • Support bind function with RTP receive hook
  • Updating the examples to use this new API
@jrsnen jrsnen added the dev Developing uvgRTP is easier, but these features are not visible to users of the library label Jul 14, 2021
@jrsnen jrsnen added this to the Release 2.2 milestone Jul 22, 2021
@jrsnen jrsnen added feature New feature and removed dev Developing uvgRTP is easier, but these features are not visible to users of the library labels Jul 22, 2021
@wowaser wowaser mentioned this issue Aug 12, 2021
Closed
@RamadanAhmed
Copy link

RamadanAhmed commented Aug 17, 2021 •
edited
Loading

Hi,
I'm very interested in the project
Can I work on this tasks if there is no assignee for it ?

@jrsnen
Copy link
Member Author

jrsnen commented Aug 23, 2021 •
edited
Loading

@RamadanAhmed Thanks for the interest. Sorry for the delay in response, I was on holiday.

Yes, you can work on this if you want. Please keep the existing C API as it is so the update doesn't break existing software. Updating the documentation would also be helpful, but I can also handle that.

@jrsnen jrsnen removed this from the Release 2.2 milestone Jul 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RamadanAhmed RamadanAhmed

Labels
feature New feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jrsnen @RamadanAhmed