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

Made some changes and updated build system #6

Closed
wants to merge 23 commits into from

Conversation

andersc
Copy link

@andersc andersc commented Jun 11, 2020

  • Print PAT/PMT information in debug mode only.
  • Created build for static library
  • Removed submodule and created external project instead (much simpler to maintain)
  • Updated readme
  • Made PAT/PMT public and created flags to see if they are valid

@andersc
Copy link
Author

andersc commented Jun 11, 2020

Added automatic build of the lib and executables.. The readme needs to be updated to match the badges on the master repository if merged.

@andersc
Copy link
Author

andersc commented Jun 12, 2020

Added prefixes following the rules ->
// Prefixes used
// m class member
// p pointer (*)
// r reference (&)
// l local scope

And changed to camelCase

@andersc
Copy link
Author

andersc commented Jun 23, 2020

The 'Multi TS-Packet demux-bug fixed' ae8b73a

Should be cherrypicked and verified.

@andersc
Copy link
Author

andersc commented Jun 23, 2020

Fully understand.. I will close this pull-request and continue working on my fork.. Will let you know if I find any bugs... (BTW .. tnks for a great project! )

@andersc andersc closed this Jun 23, 2020
@akanchi
Copy link
Owner

akanchi commented Jun 23, 2020

Thanks, and I want to synchronize your changes to autobuild and CMakeLists.txt

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.

None yet

2 participants