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

copyright pes header flag - desc 06a reserved flags fix - desc 38 and 3f support - AU_Information #25

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

nriviera
Copy link

support for copyright pes header flag

@nriviera nriviera changed the title copyright pes header flag copyright pes header flag && desc 06a reserved flags Oct 28, 2019
@nriviera nriviera changed the title copyright pes header flag && desc 06a reserved flags copyright pes header flag desc 06a reserved flags Oct 28, 2019
@nriviera nriviera changed the title copyright pes header flag desc 06a reserved flags copyright pes header flag - desc 06a reserved flags fix - desc 38 and 3f support Oct 28, 2019
@nriviera nriviera changed the title copyright pes header flag - desc 06a reserved flags fix - desc 38 and 3f support copyright pes header flag - desc 06a reserved flags fix - desc 38 and 3f support - AU_Information Oct 31, 2019
@nriviera nriviera closed this Nov 7, 2019
@nriviera nriviera reopened this Nov 7, 2019
mpeg/psi/desc_3f.h Outdated Show resolved Hide resolved
mpeg/psi/desc_38.h Outdated Show resolved Hide resolved
mpeg/psi/desc_3f.h Outdated Show resolved Hide resolved
examples/mpeg_print_au_info.c Outdated Show resolved Hide resolved
examples/mpeg_print_au_info.c Outdated Show resolved Hide resolved
examples/mpeg_print_au_info.c Outdated Show resolved Hide resolved
mpeg/au_info.h Outdated Show resolved Hide resolved
mpeg/au_info.h Outdated Show resolved Hide resolved
mpeg/pes.h Outdated Show resolved Hide resolved
mpeg/psi/desc_3f.h Outdated Show resolved Hide resolved
@nriviera
Copy link
Author

nriviera commented Feb 3, 2020

Hi, is there anything still wrong with the pull request ?
I asked for a second review but it's still waiting

@jbkempf
Copy link

jbkempf commented Feb 3, 2020

You should really rebase and squash the commits that make sense. Especially the fixup and revert ones.

@cmassiot
Copy link
Member

I agree with JB, otherwise it's good.

nriviera and others added 7 commits May 13, 2020 10:52
* Add RTP Payload Format for JPEG (rfc2435)

* dvb/si/sdt: fix consistency check between sections

Spotted by Dmitriy Novash.

* ethernet: Add 802.1q VLAN tag

* Fix ATSC desc81_bit_rate_code_from_octetrate

Co-authored-by: Josh de Kock <[email protected]>
Co-authored-by: Christophe Massiot <[email protected]>
Co-authored-by: Kieran Kunhya <[email protected]>
Co-authored-by: Arnaud de Turckheim <[email protected]>
comments

revert to include in another commit

fixed as per revision
* Add RTP Payload Format for JPEG (rfc2435)

* dvb/si/sdt: fix consistency check between sections

Spotted by Dmitriy Novash.

* ethernet: Add 802.1q VLAN tag

* Fix ATSC desc81_bit_rate_code_from_octetrate

Co-authored-by: Josh de Kock <[email protected]>
Co-authored-by: Christophe Massiot <[email protected]>
Co-authored-by: Kieran Kunhya <[email protected]>
Co-authored-by: Arnaud de Turckheim <[email protected]>
@nriviera
Copy link
Author

I did the squash and rebase.
Can you check it once more ?

@cmassiot
Copy link
Member

cmassiot commented Sep 2, 2020

Hi, the commit list seems wrong: several commits are there twice. Also there is now a conflict to fix. Can you check this? Thanks

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.

4 participants