From 9f9e720559886325b016ee84ba57d381da6611c5 Mon Sep 17 00:00:00 2001 From: Jim Stichnoth Date: Tue, 5 Mar 2013 18:52:01 -0800 Subject: [PATCH] Remove the 1088->1080 translation in the preview generator. Instead, compute the dimensions from the player's display dimension. --- mythtv/libs/libmythtv/mythplayer.cpp | 4 ++-- mythtv/libs/libmythtv/previewgenerator.cpp | 10 ---------- 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/mythtv/libs/libmythtv/mythplayer.cpp b/mythtv/libs/libmythtv/mythplayer.cpp index 5a09ddf7490..7b2b2b09e4e 100644 --- a/mythtv/libs/libmythtv/mythplayer.cpp +++ b/mythtv/libs/libmythtv/mythplayer.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); diff --git a/mythtv/libs/libmythtv/previewgenerator.cpp b/mythtv/libs/libmythtv/previewgenerator.cpp index b000da48d34..a5f002c8e62 100644 --- a/mythtv/libs/libmythtv/previewgenerator.cpp +++ b/mythtv/libs/libmythtv/previewgenerator.cpp @@ -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);