[mythtv-commits] mythtv/master commit: 857d804b8 by Mark Kendall (mark-kendall)

[MythTV]

      Author:  Mark Kendall <mkendall at mythtv.org>
 Change Date:  2011-06-20T22:50:16-07:00
   Push Date:  2011/06/20 22:50:29 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  857d804b8f57cfff800bf22866ea60480fdfb697
   Changeset:  https://github.com/MythTV/mythtv/commit/857d804b8

Log:

Store the video aspect ratio in the video buffers.

.. and use the buffer value to determine the video aspect ratio during
playback.

This has the principle (user) advantage of actually synchronising aspect
ratio changes with the actual change in the video stream (rather than
10-20 frames early).

>From a code perspective, this further reduces calls from the decoder
thread into the player, which cause significant playback interruption
with cpu based playback and introduces thread related issues for
hardware graphics contexts.

The 'video_codec_changed' parameter is removed from
MythPlayer::SetVideoParams (it has not been used since XvMC was
removed), keyframedistance is set separately and an aspect ratio change
no longer triggers a video output change directly - only when the video
frame is about to be displayed.

Theoretically there is no reason to trigger SetVideoParams for a frame
rate change either but xrandr/display res changes are triggered by a
combination of changes in video size and frame rate.

Ref #9268

Modified:

   mythtv/libs/libmythtv/avformatdecoder.cpp
   mythtv/libs/libmythtv/mythplayer.cpp
   mythtv/libs/libmythtv/mythplayer.h
   mythtv/libs/libmythtv/nuppeldecoder.cpp