-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Cea708Decoder : use good value for first range of c2 command #2595
Labels
Comments
ojw28
added a commit
that referenced
this issue
Mar 23, 2017
ojw28
added a commit
that referenced
this issue
Mar 23, 2017
Issue: #2595 ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=151004198
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hello,
We noticed a slight issue in Cea708 decoding , probably a typo/copy paste issue.
In Cea708Decoder.java :
First check/skip should be with 0x07 instead of 0x0F.
As it is described in C2 Table contains no commands as of CEA-708 revision A:
if a command is seen in these code sets a decoder must skip an appropriate number of the following bytes.
0x00-0x07 +0 bytes
0x08-0x0f +1 byte
0x10-0x17 +2 bytes
0x18-0x1f +3 bytes
Could you have a look and do the appropriate update ?
Regards
The text was updated successfully, but these errors were encountered: