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

HEVC fixes #37

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

Conversation

dantallis-edt
Copy link

Fixed some bugs affecting HEVC decode.

  • Fixed emulation prevention byte removal in the case where the first byte after an emulation prevention byte was a 0 and hence could be the start of a new sequence containing an emulation prevention byte.
  • Fixed algorithm for skipping bytes of NAL units based on nal_length. (The length already accounts for any emulation prevention bytes.)
  • Fixed processing of rbsp_trailing_bits.
  • Fixed some cases of uninitialized local variables being read.
  • Fixed some cases of missing braces on if-statement body causing unintended (although probably harmless) execution of error testing and handling.
  • Fixed some code causing compiler warnings.

@waqarz
Copy link
Contributor

waqarz commented Nov 2, 2017

Hi, what issue(s) does this PR address? also is there any test material to check its performance?

@dantallis-edt
Copy link
Author

dantallis-edt commented Nov 7, 2017 via email

@waqarz
Copy link
Contributor

waqarz commented Nov 7, 2017

It would be very relavant to create issues in this regard to see what we are fixing. 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.

2 participants