From 77173b7057e2e969ea903d1b6fc2d917ec2de9d8 Mon Sep 17 00:00:00 2001 From: Klaus Schmidinger Date: Fri, 9 Jan 2015 12:03:31 +0100 Subject: [PATCH] Fixed (well, actually worked around) a problem with subtitles not being displayed because the broadcaster doesn't set the data's version numbers as required by the DVB standard --- CONTRIBUTORS | 2 ++ HISTORY | 3 +++ dvbsubtitle.c | 46 +++++++++++++++++++++++++++++----------------- 3 files changed, 34 insertions(+), 17 deletions(-) diff --git a/CONTRIBUTORS b/CONTRIBUTORS index 3d9757e4..42ebe6b6 100644 --- a/CONTRIBUTORS +++ b/CONTRIBUTORS @@ -1190,6 +1190,8 @@ Rolf Ahrenberg registered for adding support for "Pilot", "T2-System-Id" and "SISO/MISO" parameters for increasing the PMT_SCAN_TIMEOUT to avoid timeouts when scanning PMTs + for fixing a problem with subtitles not being displayed because the broadcaster + doesn't set the data's version numbers as required by the DVB standard Ralf Klueber for reporting a bug in cutting a recording if there is only a single editing mark diff --git a/HISTORY b/HISTORY index c699cf12..664d4ab1 100644 --- a/HISTORY +++ b/HISTORY @@ -8340,3 +8340,6 @@ Video Disk Recorder Revision History (reported by Dietmar Spingler). - Fixed resetting the receiver for EMM pids for CAMs that need to receive the TS (reported by Dietmar Spingler). +- Fixed (well, actually worked around) a problem with subtitles not being displayed + because the broadcaster doesn't set the data's version numbers as required by the + DVB standard (thanks to Rolf Ahrenberg). diff --git a/dvbsubtitle.c b/dvbsubtitle.c index 92c2eb6c..d1fefdcb 100644 --- a/dvbsubtitle.c +++ b/dvbsubtitle.c @@ -7,7 +7,7 @@ * Original author: Marco Schluessler * With some input from the "subtitles plugin" by Pekka Virtanen * - * $Id: dvbsubtitle.c 3.6 2015/01/04 15:46:39 kls Exp $ + * $Id: dvbsubtitle.c 3.7 2015/01/09 11:56:25 kls Exp $ */ #include "dvbsubtitle.h" @@ -53,6 +53,8 @@ static bool DebugOutput = DebugVerbose; #define DBGMAXBITMAPS 100 // debug output will be stopped after this many bitmaps #define DBGBITMAPWIDTH 400 +#define FIX_SUBTITLE_VERSION_BROADCASTER_STUPIDITY // some don't properly handle version numbers, which renders them useless because subtitles are not displayed + // --- cSubtitleDebug -------------------------------------------------------- class cSubtitleDebug { @@ -221,8 +223,10 @@ cSubtitleClut::cSubtitleClut(int ClutId) void cSubtitleClut::Parse(cBitStream &bs) { int Version = bs.GetBits(4); +#ifndef FIX_SUBTITLE_VERSION_BROADCASTER_STUPIDITY if (clutVersionNumber == Version) return; // no update +#endif clutVersionNumber = Version; bs.SkipBits(4); // reserved dbgcluts("clut id %d version %d
\n", clutId, clutVersionNumber); @@ -355,8 +359,10 @@ cSubtitleObject::~cSubtitleObject() void cSubtitleObject::Parse(cBitStream &bs) { int Version = bs.GetBits(4); +#ifndef FIX_SUBTITLE_VERSION_BROADCASTER_STUPIDITY if (objectVersionNumber == Version) return; // no update +#endif objectVersionNumber = Version; objectCodingMethod = bs.GetBits(2); nonModifyingColorFlag = bs.GetBit(); @@ -740,8 +746,10 @@ cSubtitleRegion::cSubtitleRegion(int RegionId) void cSubtitleRegion::Parse(cBitStream &bs) { int Version = bs.GetBits(4); +#ifndef FIX_SUBTITLE_VERSION_BROADCASTER_STUPIDITY if (regionVersionNumber == Version) return; // no update +#endif regionVersionNumber = Version; regionFillFlag = bs.GetBit(); bs.SkipBits(3); // reserved @@ -849,8 +857,10 @@ void cDvbSubtitlePage::Parse(int64_t Pts, cBitStream &bs) pts = Pts; pageTimeout = bs.GetBits(8); int Version = bs.GetBits(4); +#ifndef FIX_SUBTITLE_VERSION_BROADCASTER_STUPIDITY if (pageVersionNumber == Version) return; // no update +#endif pageVersionNumber = Version; pageState = bs.GetBits(2); switch (pageState) { @@ -1387,23 +1397,25 @@ int cDvbSubtitleConverter::ExtractSegment(const uchar *Data, int Length, int64_t case DISPLAY_DEFINITION_SEGMENT: { dbgsegments("DISPLAY_DEFINITION_SEGMENT
\n"); int version = bs.GetBits(4); - if (version != ddsVersionNumber) { - bool displayWindowFlag = bs.GetBit(); - windowHorizontalOffset = 0; - windowVerticalOffset = 0; - bs.SkipBits(3); // reserved - displayWidth = windowWidth = bs.GetBits(16) + 1; - displayHeight = windowHeight = bs.GetBits(16) + 1; - if (displayWindowFlag) { - windowHorizontalOffset = bs.GetBits(16); // displayWindowHorizontalPositionMinimum - windowWidth = bs.GetBits(16) - windowHorizontalOffset + 1; // displayWindowHorizontalPositionMaximum - windowVerticalOffset = bs.GetBits(16); // displayWindowVerticalPositionMinimum - windowHeight = bs.GetBits(16) - windowVerticalOffset + 1; // displayWindowVerticalPositionMaximum - } - SetOsdData(); - ddsVersionNumber = version; - dbgdisplay("display version %d flag %d width %d height %d ofshor %d ofsver %d
\n", ddsVersionNumber, displayWindowFlag, windowWidth, windowHeight, windowHorizontalOffset, windowVerticalOffset); +#ifndef FIX_SUBTITLE_VERSION_BROADCASTER_STUPIDITY + if (version == ddsVersionNumber) + break; // no update +#endif + bool displayWindowFlag = bs.GetBit(); + windowHorizontalOffset = 0; + windowVerticalOffset = 0; + bs.SkipBits(3); // reserved + displayWidth = windowWidth = bs.GetBits(16) + 1; + displayHeight = windowHeight = bs.GetBits(16) + 1; + if (displayWindowFlag) { + windowHorizontalOffset = bs.GetBits(16); // displayWindowHorizontalPositionMinimum + windowWidth = bs.GetBits(16) - windowHorizontalOffset + 1; // displayWindowHorizontalPositionMaximum + windowVerticalOffset = bs.GetBits(16); // displayWindowVerticalPositionMinimum + windowHeight = bs.GetBits(16) - windowVerticalOffset + 1; // displayWindowVerticalPositionMaximum } + SetOsdData(); + ddsVersionNumber = version; + dbgdisplay("display version %d flag %d width %d height %d ofshor %d ofsver %d
\n", ddsVersionNumber, displayWindowFlag, windowWidth, windowHeight, windowHorizontalOffset, windowVerticalOffset); break; } case DISPARITY_SIGNALING_SEGMENT: {