[PATCHES] YaEPG Hack obsolete as of vdr-1.7.33

Zoolook · 23. Dezember 2012

Hi,

in letzter Zeit leasst mir der YaEPG-Hack irgendwie keine Ruhe, ich fing also an ihn zu jagen wo er ueberall anzutreffen ist, seit es vdr-1.7.33 gibt. Johns hat mittlerweile die API in Softhddevice im GIT, Patches fuer den eigentlichen YaEPG-HD gibt es hier, und nun habe ich mal als naechstes fuer vdr-xine-0.9.4 etwas gebastelt. Vdr-xineliboutput knuepfe ich mir auch noch demnaechst vor. Bei vdr-xine habe ich es mit dem besagtem yaepg-hd getestet, und dank des entscheidenden Hinweises von jrie auch mit nOpacity mit OSD display mode NICHT auf X11-overlay, was zwar zu lustingen Skalierungen des OSD insgesamt und je nach Sender Reduzierung der Aufloesung des OSD auf die des Videomaterials fuehrt, aber das Videofenster schien mir in allen getesteten Kombinationene von Aufloesungen und Sendeformaten mit korrektem Seitenverhaeltnis skaliert.

Werde wohl in Zukunft diesen Post solange die Patches noch noetig sind, editieren. Fuer vdr-xine-0.9.4 habe ich "for convenience" auch den noetigen PmtParser-Patch den viele kennen duerften, mit hoch geladen.

Have fun & Frohe Weihnachten, Lucian

avjui · 26. Dezember 2012

Danke für deine Mühen.

johns · 30. Dezember 2012

skinpearlhd verwendet auch -DUSE_YAEPG.

Johns

johns · 31. Dezember 2012

Also ich habe die API etwas anders verstanden.
Basierend auf Zoolook Version.

Dieser Patch:

Diff

--- yaepghd.c.orig	2012-12-31 13:30:55.901170213 +0100
+++ yaepghd.c	2012-12-31 13:36:22.060150288 +0100
@@ -162,6 +162,7 @@
 static int         iChannelNumber           = false;
 static std::string sThemeName               = "default";
 static std::string sThemeDir                = "";
+static cRect       vidWinRect               = cRect::Null;
 
 /**
  * Pirvate Classes/Function Prototypes
@@ -2633,6 +2634,7 @@
 #ifdef YAEPGHD_REEL_EHD
    reelVidWin->Close();
 #endif
+   cDevice::PrimaryDevice()->ScaleVideo(cRect::Null);
 }
 
 void
@@ -2667,7 +2669,7 @@
    /* Set up the video window parameters */
    if (VID_WIN_GEOM.w != 0 && VID_WIN_GEOM.h != 0) {
       // ask the output device to scale the video when next flushing the OSD, if it supports this
-      cRect vidWinRect(
+      vidWinRect = cRect(
          VID_WIN_GEOM.x,
          VID_WIN_GEOM.y,
          VID_WIN_GEOM.w,
@@ -3056,6 +3058,7 @@
    helpBar->Draw(mainBmp);
 
    osd->DrawBitmap(0, 0, *mainBmp);
+   cDevice::PrimaryDevice()->ScaleVideo(vidWinRect);
    osd->Flush();
 }

Alles anzeigen

CanScaleVideo prüft nur ob möglich und reduziert falls nötig das Rectangle.
ScaleVideo vor dem Flush setzt dann die Videoposition.
Beim Verlassen wird wieder die Fullscreenposition hergestellt.

Wenn dies auch schon verändert war, gibt es ein Flackern.

Anbei ein Patch für mein Plugin, die mit dem oben gepatchten yaepghd funktioniert.

Johns

kls · 31. Dezember 2012

Zitat von johns

CanScaleVideo prüft nur ob möglich und reduziert falls nötig das Rectangle.

Ich weiß ja nicht, ob du CanScaleVideo() evtl. irgendwo anders aufrufst (in dem Diff sehe ich es zumindest nicht). Es gar nicht aufzurufen ist allerdings nicht korrekt, denn es kann durchaus sein, daß ein Device nicht an jede beliebige Stelle bzw. in jeder beliebigen Größe skalieren kann. Wenn dein Plugin also allgemeingültig sein soll, dann solltest du diesen Aufruf auch machen. Im Übrigen kann es ja sogar sein, daß ein Device gar nicht skalieren kann, und evtl. sollte eine Skin da ja auch mit umgehen können.

Klaus

Uwe · 31. Dezember 2012

@Klaus: könntest du das Scale*... nicht für das hddvbdevice Plugin auch einbauen?

johns · 31. Dezember 2012

Genau, dann sieht man wie es richtig geht.

Ich dachte mir schon, daß mein Patch für Verwirrung sorgt.

Zoolock hat yaepghd bereits gepatched https://github.com/lucianm/gen…ut-YAEPG-vdr-1.7.33.patch
auf diese Änderung setzt mein Patch wiederum auf.

War zu faul, einen direkten Patch zuerzeugen.

Wenn CanScaleVideo nicht funktioniert, dann sollte ScaleVideo ja ein Noop sein.
Damit wird halt im OSD nur ein Ausschnitt des unskalierte Videos gezeigt.
Bei Yaepghd gibts (vermute ich mal) Skins die Transparent sind und ohne das Guckloch auskommen.

Johns

Zoolook · 31. Dezember 2012

Hallo,

Zitat von kls

Es gar nicht aufzurufen ist allerdings nicht korrekt, denn es kann durchaus sein, daß ein Device nicht an jede beliebige Stelle bzw. in jeder beliebigen Größe skalieren kann. Wenn dein Plugin also allgemeingültig sein soll, dann solltest du diesen Aufruf auch machen. Im Übrigen kann es ja sogar sein, daß ein Device gar nicht skalieren kann, und evtl. sollte eine Skin da ja auch mit umgehen können.

Ich hatte es so verstanden, CanScaleVideo sollte vom Skin aufgerufen werden, dann erst erfaehrt es vom Device ob und wie es skalieren kann. Ob es darauf reagiert, ist Sache des Skins, aber das Device hat es zumindest informiert, ob es das kann. Deswegen verstehe ich nicht warum das Device-Plugin CanScaleVideo aufrufen sollte, es sollte es nur zur Verfuegung stellen, dachte ich. ScaleVideo hingegen fuehrt das Skalieren bedingungslos aus, falls es das ueberhaupt kann, ansonsten passiert gar nichts, oder? Und dass es in cOsd::Flush optimalerweise aufgerufen werden sollte, sowie auch bei Schliessen des Osd, darueber hatten wir uns ja schon verstaendigt.

Guten Rutsch, Lucian

Zoolook · 31. Dezember 2012

Zitat von johns

Wenn CanScaleVideo nicht funktioniert, dann sollte ScaleVideo ja ein Noop sein.
Damit wird halt im OSD nur ein Ausschnitt des unskalierte Videos gezeigt.
Bei Yaepghd gibts (vermute ich mal) Skins die Transparent sind und ohne das Guckloch auskommen.

So funktioniert es auch, und ich denke es ist auch dem Anwender gegenueber legitim, im Falle wenn das Device nicht skalieren kann, dass ein unskalierter Bereich im Guckloch sichtbar bleibt (wie es auch urspruenglich bei nOpacity implementiert war, vor dem Patch).

Gruss, Lucian

kls · 31. Dezember 2012

Zitat von Zoolook

Hallo,

Ich hatte es so verstanden, CanScaleVideo sollte vom Skin aufgerufen werden, dann erst erfaehrt es vom Device ob und wie es skalieren kann. Ob es darauf reagiert, ist Sache des Skins, aber das Device hat es zumindest informiert, ob es das kann. Deswegen verstehe ich nicht warum das Device-Plugin CanScaleVideo aufrufen sollte,

Das habe ich ja auch nicht gesagt. Ich dachte, es geht hier um eine *Skin* - oder ist YaEPG ein Device?
Ich wollte lediglich klar machen, daß jemand, der ScaleVideo() verwendet, vorher bitteschön CanScaleVideo() aufrufen sollte.

Klaus

johns · 1. Januar 2013

yaEPG ist ein OSD Plugin, zeigt alternative EPG Ansicht.

Genau hier lagen unsere Interpretations (Zoolook + meine) Unterschiede.

Zoolook hat ScaleVideo ins cOsd::Flush() vom Ausgabeplugin eingebaut.
Und in den Destructor cOsd::cOsd() vom Ausgabeplugin.

Und ich bin der Meinung dies gehört in die Skin.

Gesundes 2013,
Johns

kls · 1. Januar 2013

Zitat von johns

yaEPG ist ein OSD Plugin, zeigt alternative EPG Ansicht.

Hmmm, ich weiß zwar nicht, was ein OSD-Plugin mit dem EPG zu tun haben sollte, aber vielleicht muß ich das auch nicht verstehen...

Zitat

Zoolook hat ScaleVideo ins cOsd::Flush() vom Ausgabeplugin eingebaut.
Und in den Destructor cOsd::cOsd() vom Ausgabeplugin.

Das ist natürlich beides falsch.

Zitat

Und ich bin der Meinung dies gehört in die Skin.

Ganz genau!

Klaus

Zoolook · 3. Januar 2013

Hi,

Zitat von johns

skinpearlhd verwendet auch -DUSE_YAEPG

ich wollte es gerade eben auch in Angriff nehmen, aber wenn Du das hier meinst, das ist kein VDR-Plugin sondern nur eine Ansammlung von Themes fuer vdr-text2skin, die optional auch YaepgHD Themes installieren. Vdr-text2skin selber scheint ueberhaupt nicht zu skalieren. Oder habe ich da was falsch verstanden?

Lucian

Zoolook · 3. Januar 2013

Zitat von kls

Hmmm, ich weiß zwar nicht, was ein OSD-Plugin mit dem EPG zu tun haben sollte, aber vielleicht muß ich das auch nicht verstehen...

Das ist natürlich beides falsch.

Ganz genau!

Klaus

Alles anzeigen

Danke Euch fuer die Erlaeuterungen, habe nun am Anfang des Threads den vdr-xine scaling Patch dem entsprechend aktualisiert, den verlinkten YaEPGHD auch.
Ciao, Lucian

johns · 4. Januar 2013

Zitat von Zoolook

Hi,

ich wollte es gerade eben auch in Angriff nehmen, aber wenn Du das hier meinst, das ist kein VDR-Plugin sondern nur eine Ansammlung von Themes fuer vdr-text2skin, die optional auch YaepgHD Themes installieren. Vdr-text2skin selber scheint ueberhaupt nicht zu skalieren. Oder habe ich da was falsch verstanden?

Ich habe nur gesucht (fgrep) wer sowas verwendet ohne darauf zuachten warum,

Johns

Zoolook · 4. Januar 2013

Zitat von johns

Ich habe nur gesucht (fgrep) wer sowas verwendet ohne darauf zuachten warum,

Habe ich mir fast gedacht, dann passt es ja, danke.

Lucian

NemoN · 4. Januar 2013

skinpearlhd (natives skin) != skin-pearlhd (text2skin)

Grüße
NemoN

Zoolook · 4. Januar 2013

Zitat von NemoN

skinpearlhd (natives skin) != skin-pearlhd (text2skin)

Danke, jetzt wo Du es sagst, erinnere ich mich es frueher mal gesehen zu haben, bloss diese Namensgebung und der Umstand dass das native Plugin fuer Gentoo nicht gepackaged war hat uns hinter's Licht gefuehrt. Ok, ich habe mal 'rein geguckt, das native skinpaerlhd verwendet (zumindest noch) keine Skalierung des Videobildes.

Gruesse, Lucian

[PATCHES] YaEPG Hack obsolete as of vdr-1.7.33

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema