[FIXED] Fortschritts- und Schnittmarkenanzeige funktioniert bei langen Aufnahmen nicht

Wenn du magst könntest du nochmal meinen Skin flatPlus ausprobieren, da dort nicht die Schnitmarken/Fortschritsanzeige vom Core verwendet werden sondern selbst gezeichnet werden. Damit könnte das Problem vielleicht weiter auf den Core eingegrenzt werden.

Grüße
Martin

Vielleicht mal mit ffmpeg prüfen, ob die tatsächliche Aufnahmelänge mit der im VDR übereinstimmt? Oder mal den index neu generieren lassen.

Das klingt für mich ja nach der cMarks Klasse konnte beim ersten rüberschauen aber nichts erkennen.

Grüße
Martin

Für Skins, die VDRs Fortschrittsanzeige verwenden, könnte das hier helfen:

--- skins.h     2013/11/03 14:07:34     3.2
+++ skins.h     2014/02/17 11:05:29
@@ -280,7 +280,7 @@
   class cProgressBar : public cBitmap {
   protected:
     int total;
-    int Pos(int p) { return p * Width() / total; }
+    int Pos(int p) { return int(int64_t(p) * Width() / total); }
     void Mark(int x, bool Start, bool Current, tColor ColorMark, tColor ColorCurrent);
   public:
     cProgressBar(int Width, int Height, int Current, int Total, const cMarks *Marks, tColor ColorSeen, tColor ColorRest, tColor ColorSelected, tColor ColorMark, tColor ColorCurrent);

Bitte mal ausprobieren.

Klaus

Danke für den Hinweis werde ich dann auch umsetzen.

Grüße
Martin

Wenn ich diesen Fix in die 2.0.6 einfließen lasse (was ich konkret vor habe), dann müsste ich fast mal APIVERSNUM hochzählen, was ja bislang noch auf 20000 steht. Allerdings würden alle Plugins auch mit dieser Änderung noch ohne Neuübersetzen laufen, da keine virtuellen Schnittstellen verändert sind. Und von dem Fix würden auch nur Skin-Plugins profitieren, neu übersetzt müssten aber *alle* werden.
Mir persönlich ist es relativ egal, aber ich möchte halt nicht unnötigen Aufwand produzieren.
Daher mal die Frage wie das hier so gesehen wird.

Klaus

Moin!

Für yaVDR ist das kein Problem, ich hab da ein Script, was der Reihe nach alle Plugins aus dem PPA herunterladen, Versionsnummer hochzählen und wieder hochladen kann. Mache ich sowieso gerne, wenn ich eine neue vdr-Version verpacke, um allen Eventualitäten aus dem Weg zu gehen.

Ob die API-Version hochgezählt wird oder nicht, ist mir relativ egal, da aber die Signatur der Funktion nicht verändert wird, müsste es meiner Meinung nach nicht unbedingt sein.

Lars.

Für Arch Linux haben wir repo-make und ein kleines Hilfsscript um die APIVERSION in alle Plugin-PKGBUILDs zu replizieren. Also auch hier kein Problem.

kls
Ich habe mir das heute noch einmal angeschaut und ich hole mir ja die Position des Markers über cMark.Position()
Dies ist ja ein int. Ich weiß nun nicht welche Werte hier auftreten können aber ist es vielleicht notwendig schon den "int position" in der cMark Klasse als int64 zu ändern? Oder können hier nicht so große Werte vorkommen? Dies hätte natürlich weitreichende Folgen innerhalb des VDRs. Ich wollte nur nachfragen, wenn jetzt schon eine Änderung gemacht werden muss ob es dann gleich komplett durchgezogen werden sollte?

Ansonsten wenn der Überlauf nur bei der Multiplikation mit der Progressbar Width vorkommt reicht natürlich die kleine Änderung.

Grüße
Martin

Die Position wird in Frames gezählt. Bei 32 Bit 'int' können das also maximal etwa 2 Milliarden Frames sein, was bei 50 fps etwas über 42 Millionen Sekunden entspricht, was wiederum knapp 12000 Stunden, also knapp 500 Tage sind. Das dürfte wohl reichen ;-).
Der Überlauf passiert echt nur kurz in der Umrechnung, denn es wird ja gleich wieder durch 'total' geteilt.

Klaus

OK danke für die Info.

Gruß
Martin