[ANNOUNCE] VDR developer version 2.3.1

@ horchi
hast du auch einen Patch für das Live Plugin und Epgsearch ?
Danke
speed

nein, aber tvguide habe ich angefangen, denke das mache ich Montag fertig

Supi !!

dann brauche ich nur noch Live und Epgsearch, dann kann ich auch endlich auf 2.3.1 umstellen.
Danke schön
speed

Mit epgsearch hab ich angefangen, ist aber eine Menge Arbeit. Hab aber gestern nach drei Stunden und schon über 1600 Zeilen diff erst mal aufgehört. Das Ding macht halt regen Gebrauch von Kanälen, Timern, Events und Aufnahmen. Verwunderlich, dass es vorher ohne Locking so gut funktioniert hat...

Lars

Danke !!

leider kenne ich mich mit C nicht aus, sonst würde ich helfen
Aber wenn erstmal alle Patche vorhanden sind können viel mehr Leute testen, und das Projekt VDR weiter voran treiben.

speed

Moin!

Die Beiträge über epgsearch hab ich mal in einen eigenen Thread ausgelagert, der später noch nach "Plugins" verschoben wird.

Lars

Moin,
hier sind die Patches für die Plugins zappilot und favorites !

Gruß
moz

cool, so langsam wirds was - danke !

Hat schon einer was für´s Live Plugin ?
Gruß
speed

Ich musste heute 40 Minuten auf den Zug warten und die Zeit irgendwie kaputt bekommen

Hab mich mal im Wiki ausgetobt: http://www.vdr-wiki.de/wiki/index.php/Plugins

(Quasi von Buchstaben zu Icons) "-" war in Tabellen eine seehehr doofe Idee.

ich denke mal so ist es übersichtlicher

(wollte nämlich schauen welche Plugins gegen 2.3.x getestet sind und die "Übersichtlichkeit" hat mich motiviert )

Edit:

Lars, TomJoad, mamomoz und horchi: vielen Dank für eure Mühen, nachmdem Klaus die neue Version mit vielen neuen Funktionen auf die Strasse gebracht hat tragt ihr maßgeblich dazu bei den aktuellen vdr development gesellschaftsfähig zu machen.

Ich denke schon sehr bald werden wir in den ersten unstable ppa anfangen können sinnvoll in der Praxis zu testen.

Vielen Dank dafür!

Christian

Zitat von Moorviper

ich denke mal so ist es übersichtlicher

(wollte nämlich schauen welche Plugins gegen 2.3.x getestet sind und die "Übersichtlichkeit" hat mich motiviert )

Könntest du noch für 'getestet - nicht funktionierend' ein anderes Icon spendieren?
Am besten ohne Häkchen, ein Strich oder durchgestrichen o.ä. Wär cool.

Und das Zahnrad bei "mit Patch" vielleicht mit weniger Zähnen. Bei der kleinen Skalierung im Wiki wird ein Kreis draus.

ja kann ich machen

evtl bei patch einfach ein Pfeil nach unten

:edit
[Blockierte Grafik: http://www.vdr-wiki.de/wiki/images/2/26/GP.png]
[Blockierte Grafik: http://www.vdr-wiki.de/wiki/images/f/f8/GX.png]

Moin,

ich weiß nicht ob noch irgendwer DVDs brennt, aber ich möchte diese Option gerne noch behalten.
Hier der Patch fürs burn plugin.

Gruß
moz

wenn du noch Lust hast wäre in der Tat das live Plugin offen genauso wie der "vdr.epgsearch-exttimeredit-0.0.2.diff" der im epgsearch code liegt. Der ist eher schwierig auf den vdr dev anzuwenden und winni wird da vermutlich auch nichts machen wollen...

Christian

Im vdr-main-loop wird eine Aufnahme gestartet mit aktiviertem TIMERS_WRITE_LOCK (vdr.c,1051), mit demselben Lock wird dann aus CRecordControls:: Start cStatus::MsgRecordings() (menu.c,5116) aufgerufen.
Als aktiver StatusMonitor bekommt auch das Plugin epgsearch die Meldung und möchte jetzt seine eigene Timerliste updaten und Konflikte auflösen.
Wie kommt das Plugin jetzt an die vdr-Timersliste? Ein Aufruf von LOCK_TIMERS_READ bzw. GetTimersRead liefert einen NULL-Pointer zurück

Spontan würde ich sagen, das Plugin müsste das in einen eigenen Thread auslagern, der dann durch den Status-Monitor getriggert wird. Sobald der dann den Lock bekommt, kann es in Ruhe arbeiten.

Lars.

Würde ich auch so sehen.

Klaus

Klaus, was hälst du von dieser Erweiterung, so dass man auch const cTimer* finden kann?
Der Inhalt von GetTimer hat ja nichts, was nicht const aufgerufen werden kann. Und es gibt manchmal Situationen, da reicht einem ein const cTimer*.

diff --git a/timers.c b/timers.c
index 771f66b..98d08d4 100644
--- a/timers.c
+++ b/timers.c
@@ -748,9 +748,9 @@ const cTimer *cTimers::GetById(int Id) const
   return NULL;
 }


-cTimer *cTimers::GetTimer(cTimer *Timer)
+const cTimer *cTimers::GetTimer(const cTimer *Timer) const
 {
-  for (cTimer *ti = First(); ti; ti = Next(ti)) {
+  for (const cTimer *ti = First(); ti; ti = Next(ti)) {
       if (!ti->Remote() &&
           ti->Channel() == Timer->Channel() &&
           (ti->WeekDays() && ti->WeekDays() == Timer->WeekDays() || !ti->WeekDays() && ti->Day() == Timer->Day()) &&
@@ -761,6 +761,11 @@ cTimer *cTimers::GetTimer(cTimer *Timer)
   return NULL;
 }


+cTimer *cTimers::GetTimer(const cTimer *Timer)
+{
+  return (cTimer*)GetTimer(Timer);
+}
+
 const cTimer *cTimers::GetMatch(time_t t) const
 {
   static int LastPending = -1;
diff --git a/timers.h b/timers.h
index 4222c10..13d3f07 100644
--- a/timers.h
+++ b/timers.h
@@ -170,7 +170,8 @@ public:
   static int NewTimerId(void);
   const cTimer *GetById(int Id) const;
   cTimer *GetById(int Id) { return const_cast<cTimer *>(static_cast<const cTimers *>(this)->GetById(Id)); };
-  cTimer *GetTimer(cTimer *Timer);
+  const cTimer *GetTimer(const cTimer *Timer) const;
+  cTimer *GetTimer(const cTimer *Timer);
   const cTimer *GetMatch(time_t t) const;
   cTimer *GetMatch(time_t t) { return const_cast<cTimer *>(static_cast<const cTimers *>(this)->GetMatch(t)); };
   const cTimer *GetMatch(const cEvent *Event, eTimerMatch *Match = NULL) const;

Lars.