Opened 11 years ago

Closed 11 years ago

#6169 closed patch (fixed)

Fix for PosMapFromEnc()

Reported by: Shane Shrybman Owned by: Shane Shrybman
Priority: minor Milestone: 0.22
Component: MythTV - Video Playback Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The PosMap? is being retrieved from the encoder but it is not appended to the decoder's PosMap?.

The biggest symptom of this is that the frontend can fall back to ffmpeg seeking during livetv or while watching an in progress recording if it attempted to seek past the end of the decoder's position map.

The patch also removes the setting of positionMapType and keyframedist from PosMapFromEnc?(). This doesn't seem to be needed and is incorrect for the playback of nuv's. Added a comment that PosMapFromDb?() must be called first to ensure these are set correctly.

Attachments (1)

fix_PosMapFromEnc.diff (1.3 KB) - added by Shane Shrybman 11 years ago.

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by Shane Shrybman

Attachment: fix_PosMapFromEnc.diff added

comment:1 Changed 11 years ago by stuartm

Owner: changed from Isaac Richards to Shane Shrybman
Status: newassigned

comment:2 Changed 11 years ago by shane

Resolution: fixed
Status: assignedclosed

(In [20191]) Closes #6169 Fixes retrieving the position map from the encoder.

Removes the setting of positionMapType and keyframedist from PosMapFromEnc?(..). Nuppeldecoder still uses MARK_KEYFRAME so the use of MARK_GOP_BYFRAME was a problem for nuv files. refs. #6170

PosMapFromDb?(..) must now be called first in order to have position map type and keyframedist set correctly.

Added a safety check to ensure keyframedist has been set before using it. refs. #6285

Note: See TracTickets for help on using tickets.