Opened 19 years ago
Closed 19 years ago
#113 closed patch (invalid)
mpeg (and other) sequence number wrap not handled
Reported by: | Owned by: | Isaac Richards | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythtv | Version: | |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
the sequence number for mpeg wraps at 33 bits which is not compensated for after timestamp normalization. this can cause audio and video timestamp mismatches and plays badly as a result. transcoding is affected too and is less recoverable.
this patch was part of the generic libavformat seek support patch but didnt get included. I feel it is important.
It offsets the pts for any format at file/stream open time to 0, thus enabling maximum uncorrupted playtime (26.5 hours for mpeg). Done this way it is transparent to transcode and will work for any format.
Equivalent compensation code is already in avformatdecoder.cpp seek which shouldnt be there if this patch is not applied.
Attachments (1)
Change History (2)
Changed 19 years ago by
Attachment: | mythtv_avf_seq.patch added |
---|
comment:1 Changed 19 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
I'd really prefer to fix this properly (ie., have the player handle the timestamp wrapping), instead of just covering it up like this. Do you have a small-ish sample of such content?