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.

Sign up for GitHub

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

Jump to bottom

TS parsing is incorrect #893

Closed
pbesedm opened this issue May 17, 2017 · 2 comments
Closed

TS parsing is incorrect #893

pbesedm opened this issue May 17, 2017 · 2 comments
Assignees
@winlinvip
Labels
Bug It might be a bug. TransByAI Translated by AI/GPT.
Milestone
2.0

Comments

@pbesedm
Copy link
Contributor

pbesedm commented May 17, 2017 •
edited by winlinvip
Loading

image

srs_kernel_ts.cpp

1658: // (1+x)B
1659: if (PES_extension_flag_2) {
1660:    PES_extension_field_length = stream->read_1bytes();
1661:    PES_extension_field_length &= 0x07;

1661 行的 PES_extension_field_length &= 0x07; 不应该是 PES_extension_field_length &= 0x7F; 吗？'

Shouldn't it be PES_extension_field_length &= 0x7F; instead of PES_extension_field_length &= 0x07; on line 1661?

TRANS_BY_GPT3
@winlinvip
Copy link
Member

👍 Yep, can you file a PR please?

@winlinvip winlinvip added the Bug It might be a bug. label May 21, 2017
@winlinvip winlinvip added this to the srs 2.0 release milestone May 21, 2017
@pbesedm pbesedm mentioned this issue May 22, 2017
Merged
@winlinvip
Copy link
Member

Merged

winlinvip added a commit that referenced this issue May 29, 2017
@winlinvip
Merge #899 to fix #893, ts PES ext length. 2.0.243
e8cb2e6
winlinvip added a commit that referenced this issue May 29, 2017
@winlinvip
For #893, #899, merge SRS2
c496648
@winlinvip winlinvip self-assigned this Sep 15, 2021
@winlinvip winlinvip changed the title TS解析不正确 TS parsing is incorrect Jul 28, 2023
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@winlinvip winlinvip

Labels
Bug It might be a bug. TransByAI Translated by AI/GPT.
Projects
None yet
Milestone
2.0
Development

No branches or pull requests
2 participants
@pbesedm @winlinvip