[Announce] recsearch - Eine Suche für die Aufnahmen

Moin!

Der Patch funktioniert soweit, ich hab das recsearch-Plugin im branch "vdr-2.1.x" mal umgestellt, so dass es den Patch voraussetzt.

Ich verstehe das Nicht-Löschen des Filters, es ist aber für recsearch schwierig herauszufinden, wann es das Objekt löschen darf. Was auch immer man von dem Filter noch brauchen mag, kann man sich eigentlich auch anders merken (recsearch braucht das Objekt nicht weiter). Und cMenuRecordings weiß, wann es den Filter nicht mehr braucht. Momentan merke ich mir den Pointer in einem statischen Feld und lösche ihn, wenn recsearch das nächste mal aufgerufen wird (in der Hoffnung, dass cMenuRecordings dann nicht mehr existiert und den Filter nicht mehr aufruft).
Ich finde es leichter, die Kontrolle über das Objekt an den vdr abzugeben. Schließlich muss man sich das cMenuRecordings-Objekt auch nicht merken, das wird ja auch vom vdr nach Benutzung wieder gelöscht.

Lars.

Nein, das ist so direkt nicht mit recsearch machbar. Der Filter wird ja immer nur auf eine Aufnahme zur Zeit angewendet.
Man könnte sicherlich die aktuelle Aufnahme gegen alle anderen testen, der Filter wird ja immer mit einem gehaltenen Lock auf die Recordings aufgerufen. Bei vielen Aufnahmen wäre das eine ziemliche Verzögerung. Hm, mal sehen.
Wie definieren sich denn doppelte Aufnahmen?

Aber vielleicht sollte einfach duplicates gefixt werden. Die Ursache für einen Absturz, weil es keine Übereinstimmungen gibt, sollte nicht so schwer zu finden sein.

Lars.

Ja ja, ich schau mal bei Gelegenheit...

Lars

Ok, danke, das werde ich mal versuchen.

Lars.

Es wäre prima, wenn du backtraces davon erzeugen könntest.
Einfach vdr-dbg und vdr-plugin-recsearch-dbg installieren, dann müssten unter /var/log/vdr coredumps und crashlogs entstehen, siehe Dokumentation.

Lars.

Zitat von kls

Dabei fällt mir noch ein: in cMenuRecordings::Open() sollte das Filter in

AddSubMenu(new cMenuRecordings(t, level + 1, OpenSubMenus, filter));

mit übergeben werden, damit es auch in einem eventuellen Untermenu funktioniert. Ein weiterer Grund, warum es nicht vom cMenuRecordings zerstört werden darf.

Überzeugt.

Lars.

Zitat von rdnzl

Leider hatte ich auch schon diverse Segfaults beim Editieren/Loschen der Suchmasken, daher möchte ich davor warnen, dies während wichtiger Aufnahmen zu tun. Sicherer/einfacher ist das Editieren der searches.conf !

Wäre prima, wenn du bei Gelegenheit backtraces liefern könntest. Ich kann mir nicht vorstellen, wo recsearch abstürzen sollte. Das würde ich gerne wissen, um es zu beheben.

Danke!
Lars.

Ja, danke, das ist schon mal ein Hinweis.
In der Tat sollen die Knöpfe da nichts tun, da scheint irgendwie das Menü darunter zu reagieren.
Jetzt habe ich einen Anhaltspunkt und auch eine Vorstellung, warum es abstürzen könnte.

Lars.

recsearch benutzt das normale Aufnahmenmenü des vdr, kein eigenes.
Wenn es das vdr-Menü kann, dann auch recsearch...
Also momentan nein, aber eine interessante Idee.

Lars.

"Menu ersetzen" heißt immer "Code aus vdr kopieren" und das bedeutet Pflegeaufwand mit jeder Version...
Ich bin froh, dass in vdr 2.1.3 der recfilter drin ist, damit man das nicht mehr machen muss.
Evtl. könnte man drüber nachdenken, die reccmds aufzubohren, so dass sie auch ohne Aufnahme aufgerufen werden können. Dann könnte man da recsearch einklinken. Ist bis jetzt aber nur eine Idee in meinem Kopf, hatte noch keine Zeit, das weiter zu verfolgen.

Lars.s

In testing gibt's eine neue Version 0.2.1, damit sollte man im OSD keinen Absturz mehr provozieren können.

Bitte testen, Danke!

Lars.