Zu lange Kurztexte (Shorttext) werden nun bei Wiedergabestatus und Info (Aufnahme/Event) gekürzt
2023-10-02: Version 0.7.3
- [Add] Cut very long shorttext in replay, recording- and event info
- [update] Removed unused icons text_new*
- [update] Remove unused icon text_timer_partial*
- [update] Replace icons text_rec_* with timerRecording_*
- [update] Use time(NULL)
- [update] Remove 'using namespace std'
- [update] Remove double var progressTop
- [update] Preincrement var 'it'Ja, läuft wie es soll.
Hab mal einen Versuch gestartet. Nicht schön:
cString info("");
if (recInfo->ShortText())
info = cString::sprintf("%s %s - %s", *ShortDateString(recording->Start()), *TimeString(recording->Start()),
recInfo->ShortText());
else
info = cString::sprintf("%s %s", *ShortDateString(recording->Start()), *TimeString(recording->Start()));
int infoWidth = fontSml->Width(info); // Width of shorttext
// TODO: Handle very long shorttext. How to get width of aspect and format icons?
// Substract 'left' in case of displayed recording icon
// Substract 'fontSmlHeight' in case of recordingerror icon is displayed later
// Substract width of aspect and format icons (ResolutionAspectDraw()) ???
int maxTextWidth = osdWidth - left - Config.decorBorderReplaySize * 2;
#if APIVERSNUM >= 20505
if (Config.PlaybackShowRecordingErrors)
maxTextWidth -= fontSmlHeight; // Substract width of imgRecErr
#endif
cImage *imgRes = imgLoader.LoadIcon("1920x1080", 999, fontSmlHeight);
maxTextWidth -= imgRes->Width() * 3; // Substract guessed max. used space of aspect and format icons
labelPixmap->DrawText(cPoint(left, fontHeight), info, Theme.Color(clrReplayFont), Theme.Color(clrReplayBg), fontSml,
maxTextWidth);
if (infoWidth > maxTextWidth) { // Add ... if info ist too long
dsyslog("flatPlus: Shorttext too long! (%d) Setting maxTextWidth to %d", infoWidth, maxTextWidth);
labelPixmap->DrawText(cPoint(left + maxTextWidth, fontHeight), "...", Theme.Color(clrReplayFont),
Theme.Color(clrReplayBg), fontSml, fontSml->Width("..."));
}Sieht dann in etwa so aus:
Zwischen Text und "..." ist noch Platz. Ich vermute, dass DrawText Buchstaben, die nicht ganz rein passen weg lässt.
Das geht doch bestimmt besser zu lösen. Aber ohne verränkungen. Die Performance dürfte bei dem Code oben aber nicht zu sehr leiden denke ich
Im skin flatPlus in displayreplay.c habe ich das Problem, dass in SetRecording der Shorttext wenn er zu lang ist die Format-Symbole überlappt, bzw. drunter duchscheit.
In der Funktion ResolutionAspectDraw() werden die Symbole ermittelt und gezeichnet.
cString info("");
if (recInfo->ShortText())
info = cString::sprintf("%s %s - %s", *ShortDateString(recording->Start()), *TimeString(recording->Start()),
recInfo->ShortText());
else
info = cString::sprintf("%s %s", *ShortDateString(recording->Start()), *TimeString(recording->Start()));
// TODO: Handle very long shorttext. How to get width of aspect and format icons?
// Substract 'left' in case of displayed recording icon
// Substract 'fontSmlHeight' in case of recordingerror icon is displayed later
// Substract width of aspect and format icons (ResolutionAspectDraw()) ???
labelPixmap->DrawText(cPoint(left, fontHeight), info, Theme.Color(clrReplayFont), Theme.Color(clrReplayBg), fontSml,
osdWidth - left - fontSmlHeight - Config.decorBorderReplaySize * 2);Mir ist auch nicht ganz klar in welcher Reihen folge das aufgerufen wird. Man (ich) könnte auch einfach schätzen wie groß die Symbole eventuell werden und den Wert mit abziehen.
Oder irgendwie über eine globale Variable. Das macht aber nur Sinn, wenn AspecrResolutionDraw() vorher aufgerufen wird
Beispiel mit Shorttext, der passt:
Rechts sind die Format Symbole
Inzwischen bei Version 0.7.2 angekommen.
2023-09-26: Version 0.7.2
- [fix] Fix display of cuttet length in replay
2023-09-25: Version 0.7.1
- [add] New symbol 'Applikationen' (MenuOrg)
- [add] Show if recording is still recording
or is replayed in menu and replaystatus
- [update] Eliminate some c-style casts
- [update] Replace some long with int32_t, etc.
- [update] Initialize some vars
- [update] Use localtime_r instead of localtime
- [update] Preincrement instead of postincrement
- [update] Remove some temporary returnvariables
2023-09-15: Version 0.7.0
- [fix] Fix wrongly named variables
- [add] From skinElchiHD - Avoid triggering index generation for recordings with empty/missing index
- [add] Add some dooutput checks
- [add] Add PixmapFill with null check
- [add] Log unknown cardsystem
- [update] Simplify GetAspectIcon and GetFormatIcon
- [update] Add more debug to CreatPixmap
- [update] Add pixmap names
- [update] Simplyfy display of crypted/uncrypted
- [update] Merge ShowRecordingErrors
- [update] Disable textscroller logging
- [update] Simplify display of radio/tv and SetMessage
2023-09-12: Version 0.6.9
- [update] Reformat some code
- [update] Use CreatePixmap() instead of osd->CreatePixmap()
- [update] Simplify check for EPGsearch searchtimermenu
- [update] Add parameter for Get...Icon()
- [update] Move PlaybackShowRecordingErrors to playback menu
- [update] Add space between Date and time in status
- [update] Rearange replay statustext for recordingerror icon at end
- [add] Add recordingerror icon to recording info header
2023-09-04: Version 0.6.8
- [add] Add option to disable recerroricons in replay
- [remove] Prevents *magick from occupying the segfaults (From SkinNopacity)
- [update] Centralize getting iconnames of format and aspect
- [update] Centralize call of pscraper plugin (From Skinnopacity)Danke, genau das dachte ich mir auch schon. Teste es dann ab Donnerstag wenn wieder Zeit ist
Noch was, das ich nicht verstehe:
Die meisten Pixmaps werden mit CreatePixmap() erzeugt. Aber in displaymenu.c gibt es einige aufrufe von
osd->CreatePixmap
Das müsste doch einfach durch CreatePixmap zu ersetzen sein? (Natürlich mit dem zusätzlichen Parameter osd)
Beispiel:
menuPixmap = osd->CreatePixmap(1, cRect(0, menuTop, menuWidth, scrollBarHeight));
zu
menuPixmap = CreatePixmap(osd, 1, cRect(0, menuTop, menuWidth, scrollBarHeight));
Hab noch ne Weile getestet und gebaut. Scheint zu laufen! Zumindest in meiner Virtualbox.
Ich denke ich werde das noch ein wenig bereinigen (Habe alten Code nur auskommentiert) und ins git stellen. Vielen Dank für die Unterstützung!
Kommen bestimmt noch mehr Fragen von mir wenn ich wieder Zeit zum Basteln habe
Schade:
g++ -Wall -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++11 -c -DPLUGIN_NAME_I18N='"skinflatplus"' -DVDRLOGO=\"vdrlogo_default\" -DWIDGETFOLDER='"/usr/lib/vdr/plugins/skinflatplus/widgets"' -I/usr/include/uuid -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/GraphicsMagick -o complexcontent.o complexcontent.c
complexcontent.c: In member function ‘void cComplexContent::CreatePixmaps(bool)’:
complexcontent.c:80:25: error: ‘osd’ was not declared in this scope; did you mean ‘Osd’?
80 | Pixmap = CreatePixmap(osd, 1, Position, PositionDraw);
| ^~~
| Osd
g++ -Wall -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++11 -c -DPLUGIN_NAME_I18N='"skinflatplus"' -DVDRLOGO=\"vdrlogo_default\" -DWIDGETFOLDER='"/usr/lib/vdr/plugins/skinflatplus/widgets"' -I/usr/include/uuid -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/GraphicsMagick -o textscroller.o textscroller.c
make[1]: *** [Makefile:102: complexcontent.o] Fehler 1
make[1]: *** Auf noch nicht beendete Prozesse wird gewartet …
textscroller.c: In member function ‘void cTextScroll::SetText(const char*, cRect, tColor, tColor, cFont*, tColor)’:
textscroller.c:18:27: error: ‘osd’ was not declared in this scope; did you mean ‘Osd’?
18 | Pixmap = CreatePixmap(osd, Layer, Position, drawPort);
| ^~~
| Osd
make[1]: *** [Makefile:102: textscroller.o] Fehler 1
make[1]: Verzeichnis „/home/darkwing/src/skin-flatplus/vdr-plugin-skinflatplus-0.6.7+git20230825-4-a2ad76d9“ wird verlassen
dh_auto_build: error: make -j4 returned exit code 2
make: *** [debian/rules:18: build] Fehler 2Da muss ich wohl auch ohne Klasse in die flat auslagern...
Vielen Dank für die Tipps. Werde ich gleich mal testen 
Das ist auch meine Intention. Aber ich bekomme das leider nicht so ohne weiteres hin. Ich dachte eigentlich dass die baserender.c mehr oider weniger der helpers in skinnopacity entspricht. Da ist das CreatPixmap ja auch implementiert, bis auf die zwei Parameter OSD und Name. Wenn ich die nicht verwende, müsste doch eigentlich alles passen? Die zwei Parameter nachzurüsten wäre aber auch nicht das Problem. Die vorkommen kann man ja suchen...
Ich würde auch gerne eine tools.[ch] erstellen um da einige Sachen auszulagern. Oder in die baserender. Das raucht mir schon die Birne. Ich schaue mir auch die Quellen von sjinnopacity usw. an für Anregungen. Aber mit dem ganzen C-Zeug stehe ich irgendwie auf Kriegsfuß
Gut, dann werde ich das wohl so lassen. Ist nur blöd bei Änderungen; die muss man dann drei mal machen
shellcheck sagt:
Line 7:
if [ ! -e $TOGGLE* ]; then
^-- SC2144 (error): -e doesn't work with globs. Use a for loop.Mit einer for-Schleife müsste das ganz gut gehen
Hab alle ausse in baserender auskommentiert. Klappt natürlich nicht:
g++ -Wall -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++11 -c -DPLUGIN_NAME_I18N='"skinflatplus"' -DVDRLOGO=\"vdrlogo_default\" -DWIDGETFOLDER='"/usr/lib/vdr/plugins/skinflatplus/widgets"' -I/usr/include/uuid -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/GraphicsMagick -o textscroller.o textscroller.c
complexcontent.c: In member function ‘void cComplexContent::CreatePixmaps(bool)’:
complexcontent.c:80:12: error: ‘CreatePixmap’ was not declared in this scope; did you mean ‘CreatePixmaps’?
80 | Pixmap = CreatePixmap(1, Position, PositionDraw);
| ^~~~~~~~~~~~
| CreatePixmaps
make[1]: *** [Makefile:102: complexcontent.o] Fehler 1
make[1]: *** Auf noch nicht beendete Prozesse wird gewartet …
textscroller.c: In member function ‘void cTextScroll::SetText(const char*, cRect, tColor, tColor, cFont*, tColor)’:
textscroller.c:18:14: error: ‘CreatePixmap’ was not declared in this scope
18 | Pixmap = CreatePixmap(Layer, Position, drawPort);
| ^~~~~~~~~~~~
make[1]: *** [Makefile:102: textscroller.o] Fehler 1
make[1]: Verzeichnis „/home/darkwing/src/skin-flatplus/vdr-plugin-skinflatplus-0.6.7+git20230825-4-a2ad76d9“ wird verlassen
dh_auto_build: error: make -j4 returned exit code 2
make: *** [debian/rules:18: build] Fehler 2
dpkg-buildpackage: Fehler: Unterprozess debian/rules build lieferte Exitstatus 2Sind alle gleich. Am Ende:
return osd->CreatePixmap(Layer, ViewPort, SafeDrawPort);Mal wieder eine Frage von einem unwissenden…
Bei dem skin flatPlus ist mir im Quellcode aufgefallen, dass CreatePixmap dreim al vorhanden ist und mit dem selben Code gefüllt ist. In baserender .h unter public: und die anderen beiden .h unter private:
Ich frage mich nun nach dem Sinn des ganzen. Wäre es nicht einfacher das nur in einer .h zu haben?
Wer kennt sich da aus. Wie kompliziert wäre das?
Habe ein neues Release gemacht: https://github.com/MegaV0lt/vd…atplus/releases/tag/0.6.7
Code
2023-08-24: Version 0.6.7
- [fix] Adaptation to tvscraper commit a4287558
- [update] Prevents *magick from occupying the segfaults (From SkinNopacity)Hab ein wenig bei SkinNopacity ab geschaut
So. Hab auch ein wenig gebastelt und ein neues Release gemacht: https://github.com/MegaV0lt/vd…atplus/releases/tag/0.6.7
2023-08-24: Version 0.6.7
- [fix] Adaptation to tvscraper commit a4287558
- [update] Prevents *magick from occupying the segfaults (From SkinNopacity)Hab ein wenig bei SkinNopacity ab geschaut
Ich baue das gerne um. Sollte ja nicht schwer das zu ersetzen. Schau morgen wenn ich zu hause bin ob ich es hin bekomme.
Ok, Darsteller sind in einem extra Tab. Sind also in Live vorhanden. Was stimmt da im Skin nicht, bzw was hat sich geänbdert. Hier der code aud dispayreplay.c, der das Banner anzeigen soll:
std::string mediaPath("");
int mediaWidth = 0;
int mediaHeight = 0;
// TVScraper
// first try scraper2vdr
static cPlugin *pScraper = cPluginManager::GetPlugin("scraper2vdr");
if (!pScraper) // if it doesn't exit, try tvscraper
pScraper = cPluginManager::GetPlugin("tvscraper");
if (Config.TVScraperReplayInfoShowPoster && pScraper) {
ScraperGetEventType call;
call.recording = recording;
int seriesId = 0;
int episodeId = 0;
int movieId = 0;
if (pScraper->Service("GetEventType", &call)) {
seriesId = call.seriesId;
episodeId = call.episodeId;
movieId = call.movieId;
}
if (seriesId > 0) {
cSeries series;
series.seriesId = seriesId;
series.episodeId = episodeId;
if (pScraper->Service("GetSeries", &series)) {
if (series.banners.size() > 0) {
mediaPath = series.banners[0].path;
mediaWidth = series.banners[0].width * Config.TVScraperReplayInfoPosterSize * 100;
mediaHeight = series.banners[0].height * Config.TVScraperReplayInfoPosterSize * 100;
}
}
} else if (movieId > 0) {
cMovie movie;
movie.movieId = movieId;
if (pScraper->Service("GetMovie", &movie)) {
mediaPath = movie.poster.path;
mediaWidth = movie.poster.width * 0.5 * Config.TVScraperReplayInfoPosterSize * 100;
mediaHeight = movie.poster.height * 0.5 * Config.TVScraperReplayInfoPosterSize * 100;
}
}
}Bei Filmen wir alles angezeigt, bei Serien nichts 