Closed
Bug 757600
Opened 12 years ago
Closed 12 years ago
Ogg Opus header version bump
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: rillian, Assigned: rillian)
Details
Attachments
(1 file, 1 obsolete file)
2.64 KB,
patch
|
rillian
:
review+
|
Details | Diff | Splinter Review |
The Ogg Opus header had a version bump recently, with no other changes. We're now supposed to accept any stream with version > 16, i.e. when the high four bits are zero.
Comment 1•12 years ago
|
||
(In reply to Ralph Giles (:rillian) from comment #0) > We're now supposed to accept any stream with version > 16, i.e. when the < 16, to be clear.
Assignee | ||
Updated•12 years ago
|
Whiteboard: [autoland-try:-b do -p all -u mochitest-1 -t none]
Assignee | ||
Updated•12 years ago
|
Attachment #626170 -
Flags: review?(cpearce)
Updated•12 years ago
|
Whiteboard: [autoland-try:-b do -p all -u mochitest-1 -t none] → [autoland-in-queue]
Comment 3•12 years ago
|
||
Comment on attachment 626170 [details] [diff] [review] Update the Opus version field parser Review of attachment 626170 [details] [diff] [review]: ----------------------------------------------------------------- Since mVersion is only used once, just store it in a local variable rather than adding a datafield to nsOpusState. r+ with that change.
Attachment #626170 -
Flags: review?(cpearce) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Updated patch addressing review comments.
Attachment #626247 -
Flags: review+
Assignee | ||
Updated•12 years ago
|
Attachment #626170 -
Attachment is obsolete: true
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f93a650e8a8c
Flags: in-testsuite-
Keywords: checkin-needed
Whiteboard: [autoland-in-queue]
Target Milestone: --- → mozilla15
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f93a650e8a8c
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•