Plugin-Patches für vdr-1.7.32

iNOB · Nov 30th 2012

Hmm... mit dem Patch bekomme ich bei burn, xine und xinelibout nen Fehler:

burn:

Code

scanner.c:408: error: ‘class cPatPmtParser’ has no member named ‘PmtPid’
scanner.c:413: error: ‘class cPatPmtParser’ has no member named ‘PmtPid’

xine:

Code

xineDevice.c:1203: error: ‘class cPatPmtParser’ has no member named ‘PmtPid’

xinelibout:

Code

device.c:1162: error: ‘const class cPatPmtParser’ has no member named ‘PmtPid’

Edit: Hab die entsprechenden Stellen an den Patch angepasst, jetzt gehts wieder.

Gruß
iNOB

:modON
abgetrennt vom Thread
Neue Schnittfunktion in VDR 1.7.32

nvertigo · Nov 30th 2012

Hi iNOB,

sitze gerade ab burn. Zeile 408 ist klar, aber mit der logging-Ausgabe in 413 tue ich mich schwerer. Wie hast Du das gelöst?

Gruß, Ingo

iNOB · Nov 30th 2012

Groß- und Kleinschreibung beachten

Gruß
iNOB

nvertigo · Nov 30th 2012

...stehe gerade echt auf dem Schlauch. Das, was vorher in PmtPid stand, finde ich jetzt nur noch privat in PatPmtParser.pmtPids[0].... Aber c++ Nachhilfe ist wohl ziemlich OT hier.

Gruß, ingo

kls · Nov 30th 2012

Quote from nvertigo

...stehe gerade echt auf dem Schlauch. Das, was vorher in PmtPid stand, finde ich jetzt nur noch privat in PatPmtParser.pmtPids[0]....

Das, was bisher

Code

if (Pid == PatPmtParser.PmtPid())

hieß muß jetzt

Code

if (PatPmtParser.IsPmtPid(Pid))

heißen, damit auch Aufnahmen von anderen Receivern abgespielt werden können, die PATs enthalten, die mehrere PMT-PIDs haben.

Aber in erster Linie geht es hier und jetzt darum, die neue Schnittfunktion zu testen

Klaus

nvertigo · Nov 30th 2012

...das ist Zeile 408. Ich verzweifle an

Code

logger::debug(format( "PID found: PMT PID=0x{0}, Vpid=0x{1}, Vtype=0x{2}") % format::base( PatPmtParser.PmtPid(), 16 ) % format::base( PatPmtParser.Vpid(), 16 ) % format::base( PatPmtParser.Vtype(), 16 ));

Will am Wochenende ausführliche Tests zum Schneiden machen.

Gruß, Ingo

iNOB · Nov 30th 2012

burn (scanner.c)

Code

@@ Zeile 409 @@
-					else if (Pid == PatPmtParser.PmtPid())
+					else if (PatPmtParser.IsPmtPid(Pid))
@@ Zeile 414 [EDIT]@@
-						logger::debug(format( "PID found: PMT PID=0x{0}, Vpid=0x{1}, Vtype=0x{2}") % format::base( PatPmtParser.PmtPid(), 16 ) % format::base( PatPmtParser.Vpid(), 16 ) % format::base( PatPmtParser.Vtype(), 16 ));
+						// logger::debug(format( "PID found: PMT PID=0x{0}, Vpid=0x{1}, Vtype=0x{2}") % format::base( PatPmtParser.PmtPid(), 16 ) % format::base( PatPmtParser.Vpid(), 16 ) % format::base( PatPmtParser.Vtype(), 16 ));

xine (xineDevice.c)

Code

@@ Zeile 1203 @@
-      else if (pid == patPmtParser.PmtPid())
+      else if (patPmtParser.IsPmtPid(pid))

xineliboutput (device.c)

Code

@@ Zeile 1162 @@
-          if (Pid == 0 || Pid == PatPmtParser()->PmtPid()) {
+          if (Pid == 0 || PatPmtParser()->IsPmtPid(Pid)) {

Keine Ahnung ob das korrekt ist. Aber es compiliert und der VDR liefert mir ein Bild. Denke mal das es passt.

Gruß
iNOB

nvertigo · Nov 30th 2012

...aber "PatPmtParser.IsPmtPid(Pid)" liefert doch nur true oder false, aber nicht Pid des letzten Pmt... oder?

Gruß, Ingo

M-Reimer · Nov 30th 2012

Stimmt. Folglich ist sein Fix für die "logger"-Zeile so wenig sinnvoll. Ich würde die Zeile schlicht raushauen Wobei mich ohnehin wundert, dass noch jemand Aufnahmen wegbrennt...

iNOB · Nov 30th 2012

Quote from Mreimer

Wobei mich ohnehin wundert, dass noch jemand Aufnahmen wegbrennt

Restbestände die aus historischen Gründen einfach mitcompiliert werden. Benutzt hab ich burn in den letzten 10 Jahren vielleicht 2x. Guter Tip das logging an der Stelle komplett wegzuhauen. Habs oben diesbezüglich korrigiert.

Gruß
iNOB

M-Reimer · Nov 30th 2012

Vielleicht ist in dem Fall der Tipp, das Burn-Plugin wegzuhauen, sogar noch sinnvoller

geronimo · Nov 30th 2012

Quote

Wobei mich ohnehin wundert, dass noch jemand Aufnahmen wegbrennt...

Ihr vergisst - es gibt noch viele Quadratschädel ohne VDR.
Wenn ich z.B. meiner Mutter was zum Anschauen geben will, dann geht das nur per DVD.

... und wenn ich nicht aufgepasst und den Film in HD aufgenommen habe, dann heißt das eben runter-rechnen, umwandeln und brennen
Also bei mir läuft der DVD-Brenner noch regelmäßig.

Gruß Gero

P.S. ok, ohne burn-Plugin kann ich gut leben. Ich verarbeite die Filme dezentral.

Keine_Ahnung · Nov 30th 2012

Quote from Mreimer

Wobei mich ohnehin wundert, dass noch jemand Aufnahmen wegbrennt...

Darum gehts ja nicht. Wenn man einfach mal ne Aufnahme irgendwohin mitnehmen will oder jemand einen bittet was für ihn aufzunhmen ist es einfach praktisch.

cu

iNOB · Nov 30th 2012

Hab eigentlich gedacht, dass es hier lediglich um die neue Schnittfunktion geht. Also bitte zurück zum Thema.

Gruß
iNOB

FireFly · Nov 30th 2012

Quote from Mreimer

Vielleicht ist in dem Fall der Tipp, das Burn-Plugin wegzuhauen, sogar noch sinnvoller

Danke für die Blumen. Dafür habe ich also dutzende Wochenenden damit verbracht, um das Burn-plugin für TS kompatibel zu machen. Werde mich gelegentlich bei Dir revanchieren ....

Die Änderungen für burn im Post 58 sind übrigens funktional so korrekt.

M-Reimer · Nov 30th 2012

Wir weichen von dem Sinn des Threads ab... Klaus wollte Tester für seine neue Schnittfunktion finden...

Quote from FireFly

Dafür habe ich also dutzende Wochenenden damit verbracht, um das Burn-plugin für TS kompatibel zu machen. Werde mich gelegentlich bei Dir revanchieren ....

So war das nicht gemeint. Ich habe nur zwischen den Zeilen gelesen, dass hier ein Plugin mitgezogen wird, das garnicht mehr genutzt wird.

Nachtrag: Und für den Fall, dass du dich persönlich angegriffen gefühlt hast, bitte ich um Entschuldigung.

FireFly · Nov 30th 2012

Quote from Mreimer

Ich habe nur zwischen den Zeilen gelesen, dass hier ein Plugin mitgezogen wird, das garnicht mehr genutzt wird.

Naja, einige nutzen es noch und ich versuche es zu pflegen, aber ich selbst nutze es auch nicht mehr. Da heute ja schon "jedes" Smartphone streamen kann, werden DVDs halt deutlich seltener benötigt.
Sorry für meine Überreaktion.

jsffm · Nov 30th 2012

gelöscht.

Keine_Ahnung · Nov 30th 2012

Quote from jsffm

FireFly

könnt Ihr das nicht in nem andren Thread klären? Der Thread verliert total die Übersichtlichkeit!

War doch schon längst alles erledigt, jetzt warten wir nur noch drauf das die (üblicherweise 5-10) Leute noch die OT Postings nachschieben in denen sie sagen das OT Postings den Thread kaputtmachen. Aber wenn die durch sind ist wieder Ruhe.

SCNR

Teckelberry · Feb 28th 2013

Tag

bei mir bauen burn und xine trotz der obigen Änderungen nicht durch. Ich habe Gen2VDR V3 incl allen Updates installiert.

Kann mir jemand helfen?

Plugin-Patches für vdr-1.7.32

Participate now!

Share