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

Rtp h263 test and fix #119

Merged
merged 6 commits into from
Aug 8, 2022
Merged

Conversation

ManishaJajoo
Copy link
Contributor

No description provided.

Change-Id: I43dfbabcbe686c31cb54e6b95688af1fa35a3d24
Change-Id: I57d57881ef5c158d41be1bf1e3714332d50cd3a9
Earlier, the consume method of RtpH263Reader was changing the bytes of the
input bitstream during header parse. This commit copies the input into
local context and changes the local variable as per the specifications
thus keeping the input constant.
This change is done to keep the frame data unchanged.
RtpH263Reader changes the header data in input, so to send the same
RTP packet across multiple tests, each test copies the frame data
into a new packet and sends that to the reader.
@claincly
Copy link
Contributor

Merged internally.

@tonihei tonihei merged commit e54d2f5 into androidx:main Aug 8, 2022
marcbaechinger pushed a commit that referenced this pull request Sep 30, 2022
microkatz pushed a commit that referenced this pull request Nov 22, 2022
PiperOrigin-RevId: 463146426
(cherry picked from commit e54d2f5)
icbaker pushed a commit that referenced this pull request Dec 12, 2022
PiperOrigin-RevId: 463146426
(cherry picked from commit 92fc065)
@androidx androidx locked and limited conversation to collaborators Mar 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants