X Tutup
The Wayback Machine - https://web.archive.org/web/20200916121053/https://github.com/niqdev/ipcam-view/issues/32
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

Wrap android-mjpeg-view project #32

Open
niqdev opened this issue Mar 24, 2017 · 4 comments
Open

Wrap android-mjpeg-view project #32

niqdev opened this issue Mar 24, 2017 · 4 comments
Labels

Comments

@niqdev
Copy link
Owner

@niqdev niqdev commented Mar 24, 2017

@perthcpe23 I really like your clear and simple implementation in your project and I would like to initially wrap it and if/when stable maybe replace the other 2 libraries with yours and bump it to version 2.

@perthcpe23
Copy link

@perthcpe23 perthcpe23 commented Dec 1, 2018

Hi, I know it has been too long. Sorry for that. I recently updated the library (detect a boundary from HTTP header) and a license (MIT) which means your idea about wrapping a library is definitely OK. I hope this can help. Thanks!

@niqdev
Copy link
Owner Author

@niqdev niqdev commented Dec 2, 2018

Hi, thanks for your interest!

It's hard to maintain the current wrapped libraries and one possible idea would be to rewrite it from scratch in Kotlin as mentioned in #59 and use only the most reliable implementation.

I doubt it will ever happen any time soon, but I think we should keep your implementation in mind as well.

By the way, let me know if you have interest instead in adding your impementation now as intermediate step, keeping in mind the above.

Thanks

@perthcpe23
Copy link

@perthcpe23 perthcpe23 commented Dec 3, 2018

Hi, +1 for reimplementing library in Kotlin. I'm using Kotlin so I definitely can contribute. Intermediate step also sounds good to me. Please give me a week or two I will fork the project and try to add my library in to it.

@niqdev
Copy link
Owner Author

@niqdev niqdev commented Dec 3, 2018

Thanks and you are more than welcome to contribute to the project 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.
X Tutup