Permalink
Browse files

Remove the 1088->1080 translation in the preview generator.

Instead, compute the dimensions from the player's display dimension.
  • Loading branch information...
stichnot authored and jyavenard committed Mar 6, 2013
1 parent b38d5df commit 2f0cfb811b386616a1af844c6633fa69edb3e480
Showing with 2 additions and 12 deletions.
  1. +2 −2 mythtv/libs/libmythtv/mythplayer.cpp
  2. +0 −10 mythtv/libs/libmythtv/previewgenerator.cpp
@@ -4344,8 +4344,8 @@ char *MythPlayer::GetScreenGrabAtFrame(uint64_t frameNum, bool absolute,
&retbuf, PIX_FMT_RGB32, &orig, PIX_FMT_YUV420P,
video_dim.width(), video_dim.height());
vw = video_dim.width();
vh = video_dim.height();
vw = video_disp_dim.width();
vh = video_disp_dim.height();
ar = frame->aspect;
DiscardVideoFrame(frame);
@@ -548,16 +548,6 @@ bool PreviewGenerator::SavePreview(QString filename,
if (!data || !width || !height)
return false;
if( height == 1088 )
{
// Remove the extra 8 pixels at the bottom of 1080i recordings that
// decode to 1088 rows as these are bogus pixels and make the previews
// look a bit wrong. The worst offender seems to be H.264 captures
// from HDPVR. Apparently, BBC HD also may exhibit the same behavior
// in the UK, although with a different width.
height = 1080;
}
const QImage img((unsigned char*) data,
width, height, QImage::Format_RGB32);

0 comments on commit 2f0cfb8

Please sign in to comment.