skin-flatplus - Sonderzeichen in Aufnahmemenü

Hmm habe ich dort auch noch nicht gesehen. Welche Schriftart hast du denn eingestellt? Und welche Locale hast du im Linux eingestellt?

Grüße
Martin

Ich habe hier genau das gleiche Problem mit skinflatplus unter Archlinux und hatte mir das Problem schon vor Wochen mal etwas genauer angesehen. Bis jetzt kam ich aber noch nicht dazu das Problem im Bugracker eintragen oder gar einen richtigen Patch zu basteln. (Schande über mein Haupt... )

Das Problem tritt in der Funktion GetRecordingName auf, welche die Schnittzeichen und Scrolltext (?) erstellt. Kommentiere ich diesen Teil aus und gebe einfach nur den Aufzeichnungsnamen aus, funktioniert es perfekt mit der Einschränkung fehlender Lauftexte und "%"-Zeichenentfernung.
Ich hatte vermutet, dass es es mit dem in GCC 5.1 neu eingeführten std:string Handling zu tun haben könnte, da es nach irgend einem Systemupdate von Archlinux das erste mal aufgetreten ist. Weiter verfolgt habe ich es allerdings aus Zeitproblemen bisher nicht. Hast du hier eine Idee, Martin? Das Problem tritt übrigens unabhängig von der verwendeten Schriftart auf.

Grüße,
Markus

--- displaymenu.orig.c	2016-05-05 21:30:58.000000000 +0200
+++ displaymenu.c	2016-05-06 23:26:32.000000000 +0200
@@ -3977,26 +3977,7 @@
 const char * cFlatDisplayMenu::GetRecordingName(const cRecording *Recording, int Level, bool isFolder) {
     if( !Recording )
         return "";
-    std::string recNamePart;
-    std::string recName = Recording->Name();
-    try {
-        std::vector<std::string> tokens;
-        std::istringstream f(recName.c_str());
-        std::string s;
-        while (std::getline(f, s, FOLDERDELIMCHAR)) {
-            tokens.push_back(s);
-        }
-        recNamePart = tokens.at(Level);
-    } catch (...) {
-        recNamePart = recName.c_str();
-    }
-
-    if( Config.MenuItemRecordingClearPercent && isFolder ) {
-        if( recNamePart[0] == '%' ) {
-            recNamePart.erase(0, 1);
-        }
-    }
-    return recNamePart.c_str();
+    return Recording->Name();
 }
 
 const char* cFlatDisplayMenu::GetGenreIcon(uchar genre) {

Hi,

ich habe die Zeit gefunden mir das Problem einmal genauer anzusehen und einen Patch (siehe Anhang) zu basteln. Wäre schön wenn Martin diesen ins Git aufnehmen könnte.

Grüße,
Markus

Diese komischen Zeichen im Screenshot des ersten Post deuten immer darauf hin, dass da Pointer auf Strings benutzt werden, die dann nicht mehr gültig sind.
Wenn man Glück hat, dann kommt da an früher Stelle eine Null und man hat nur ein paar komische Zeichen. Normalerweise sollte das Programm abstürzen, weil es auf nicht initialisierten oder schon wieder freigegebenen Speicher zugreift.

Lars.

Und wenn ich mir den Patch anschaue, ist genau das passiert.
Innerhalb der Funktion wird die Variable recNamePart definiert. Und es wird dann ein Pointer auf diese Variable (bzw. dessen Inhalt) zurückgegeben. Und direkt bei der Rückgabe wird diese Variable natürlich zerstört.

Lars.

Zitat von Delekhan

Wäre schön wenn Martin diesen ins Git aufnehmen könnte

Am besten wäre ein Eintrag im Bugtracker: https://projects.vdr-developer…s/plg-skinflatplus/issues

Ich habe den Thread leider etwas übersehen. Es wäre bei sowas immer gut in im git ein Ticket aufzumachen, dann bekomme ich ne Info per Mail

Wenn der Patch funktioniert, dann super dann werde ich den mit aufnehmen. Mir ist wie gesagt nie etwas aufgefallen aber so wie der Patch aussieht habe da wohl wirklich was Speichertechnisch falsch gemacht.
Ich werde den mal 1:1 übernehmen und bei mir nochmal testen und dann ins git aufnehmen. Danke!

Grüße
Martin

Ich kenne diese Anzeigeprobleme auch nicht. Vermutlich tritt das nur in bestimmten Konfigurationen auf

Wenn es nicht auftritt, ist das reines Glück, der Fehler ist definitiv da.

Lars

Habs endlich geschafft und der fix ist im git

Kommt da noch mehr oder soll ich das schon mal für yaVDR verpacken?

Also ich habe gerade einiges im git aktualisiert und mein Changelog zur vorherigen Version ist relativ lang. Ich würde jetzt eine oder zwei Wochen auf Feedback warten und dann eine neue "stabile" Version 0.5.2 rausbringen.
Also wenn du möchtest kannst du gerne verpacken ansonsten noch 2 Wochen warten und dann gibt es ne neue Version

Grüße
Martin

Dann kommt es erst mal ins testing-vdr PPA für yaVDR 0.6