[vdr] [ANNOUNCE] VDR developer version 1.7.21

[Luca Olivetti]

Al 09/09/11 16:54, En/na Klaus Schmidinger ha escrit:

>> I think this is still needed for output plugins that don't implement the method,
>> so that they should at least work as before this modification.
> 
> Can you please point out exactly which modificaton you are
> referring to?

I'm referring to the modification that changed GetVideoSize
from being purely informational to being used for something.
In case you're interested, with the patch below, output plugins that
don't implement GetVideoSize will have the subtitles working as
before. I know that the proper fix is to implement GetVideoSize 
and GetOsdSize (I actually did afterwards), but at least it doesn't
break what was working before.

--- dvbsubtitle.c.orig	2011-09-04 19:11:12.426133000 +0200
+++ dvbsubtitle.c	2011-09-09 19:27:46.064725000 +0200
@@ -887,7 +887,7 @@
   double VideoAspect;
   cDevice::PrimaryDevice()->GetOsdSize(OsdWidth, OsdHeight, OsdAspect);
   cDevice::PrimaryDevice()->GetVideoSize(VideoWidth, VideoHeight, VideoAspect);
-  if (OsdWidth == displayWidth && OsdHeight == displayHeight) {
+  if ((OsdWidth == displayWidth && OsdHeight == displayHeight) || VideoWidth == 0) {
      osdFactorX = osdFactorY = 1.0;
      osdDeltaX = osdDeltaY = 0;
      }


Bye
-- 
Luca