Re: [ANNOUNCE] VDR developer version 1.7.21

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
vdr mailing list
vdr@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


[Index of Archives]     [Linux Media]     [Asterisk]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Big List of Linux Books]     [Fedora Users]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux