Tv-Onscreen 0.7.1

BEEjay

Intermediate

Posts: 453

Location: Kirmes am Kanale grande!

Occupation: Programmierer

1

Sunday, January 9th 2005, 5:42pm

Die Version sollte nun auch mit der neuen VDR Version funktionieren

http://beejay.vdr-developer.org/patches/…en-0.7.1.tar.gz

Das ist mein erster Versuch nen Plugin zu patchen, hoffe das es wirklich überall funktioniert

Thx @ Thomas für den Webspace

Freu mich über Feedback

www.sascha-bajonczak.de

Go to the top of the page

AnK

Master

Posts: 2,389

Location: DE

2

Sunday, January 9th 2005, 6:04pm

Hast du im Vergleich zum *.diff aus dem "VDR 1.3.18" Thread sonst noch etwas verändert?

HW I: Thermaltake DH 202, Intel DH67BL, Intel Pentium G630, TT Premium S2-6400, Crucial m4 SSD,
Fortron Source Aurum Gold 400W, Scythe Big Shuriken, LG DVD-RW GH22NS50, Kingston ValueRAM 4GB
HW II: A+case Cupid 2, Intel D945GSEJT, 512 MB RAM, 2x PCI Riser, Atric IR-Einschalter Rev. 5,
Hauppauge Nexus Rev 2.2, 60W 12V/DC, Slimline Samsung DVD-RW, CF/SATA-Adapter, 4 GB CF Card

Go to the top of the page

apex

Intermediate

Posts: 577

3

Sunday, January 9th 2005, 6:11pm

@ beejay

und das läuft bei dir? Ich habe stark verwürfelte Schriftzeichen wenn ich TVOnscreen aufrufe. Man kann es kaum lesen.

Go to the top of the page

sundin

Intermediate

Posts: 403

Location: München

4

Sunday, January 9th 2005, 6:21pm

RE: Tv-Onscreen 0.7.1

Quoted

Original von BEEjay
Freu mich über Feedback

Statt

	Source code
1 2	cReadLine *FileOps= new cReadLine(); while ((s = FileOps->Read(f)) != NULL)

besser

	Source code
1 2	cReadLine FileOps; while ((s = FileOps.Read(f)) != NULL)

Sonst hast du ein klassisches Speicherleck.

Ich weiss auch nicht, ob der Plugin-Maintainer das gern hat, wenn du
ein "inoffizielles 0.7.1" rausbringst, das will er wahrscheinlich selber
in der Hand haben

Grüße
Achim

my VDR: Asus AT3IONT-I deluxe, Digital Devices Cine S2 (dual DVB-S2 V5.5), yavdr 0.4, VDR 1.7.21
OSD-Calender: PIM Plugin

Go to the top of the page

apex

Intermediate

Posts: 577

5

Sunday, January 9th 2005, 6:32pm

bei mir sieht das OSD so aus (auch nach obiger korrektur)

apex has attached the following image:

This post has been edited 1 times, last edit by "apex" (Jan 9th 2005, 6:33pm)

Go to the top of the page

Brougs78

Master

Posts: 2,450

Location: Tirol (A)

6

Sunday, January 9th 2005, 6:41pm

RE: Tv-Onscreen 0.7.1

Quoted

Original von sundin
Ich weiss auch nicht, ob der Plugin-Maintainer das gern hat, wenn du
ein "inoffizielles 0.7.1" rausbringst, das will er wahrscheinlich selber
in der Hand haben

Yepp, das finde ich auch, dass das nicht ganz glücklich gewählt ist. Auch vdr-burn-0.0.6 (anderer Thread) ist nicht so ideal, da es dann - bei einem wirklichen Release der Version - Verwirrungsgefahr gibt.

Trotzdem danke, dass du hier Patches uploadest.

Gruß,
Brougs78

- -- --- ================================================================ --- -- -
VDR-Konfig: Antec Fusion, Intel E5200, Asus P5N7A-VM (VDPAU), 2xDVB-s2 // yavdr-0.2
- -- --- ================================================================ --- -- -

» Mein VDR-Zeugs (OSDImage-Plugin, Enigma-Skin): brougs78.vdr-developer.org «

Go to the top of the page

apex

Intermediate

Posts: 577

7

Sunday, January 9th 2005, 6:50pm

bei euch geht das mit dem patch für tvonscreen? oder sieht eure anzeige auch so aus wie meine?

Go to the top of the page

sundin

Intermediate

Posts: 403

Location: München

8

Sunday, January 9th 2005, 7:01pm

Quoted

Original von apex
bei euch geht das mit dem patch für tvonscreen? oder sieht eure anzeige auch so aus wie meine?

Bei mir siehts anders aus:

sundin has attached the following image:

my VDR: Asus AT3IONT-I deluxe, Digital Devices Cine S2 (dual DVB-S2 V5.5), yavdr 0.4, VDR 1.7.21
OSD-Calender: PIM Plugin

Go to the top of the page

BEEjay

Intermediate

Posts: 453

Location: Kirmes am Kanale grande!

Occupation: Programmierer

9

Sunday, January 9th 2005, 7:02pm

Hi Sorry wie gesagt sind meine ersten versuche gut das mit der Verisonsnummer ist unglücklich gewählt aber werd in zukunft lieber unterlassen

Ich konnte es selber noch nciht testen weil cih grade auch was aufnehme aber sonst wurde bisher nicht viel geändert daher könnte ich das nicht nachvollziehen was dort die schrift etwas überscheniden lässt @ apex

www.sascha-bajonczak.de

Go to the top of the page

Brougs78

Master

Posts: 2,450

Location: Tirol (A)

10

Sunday, January 9th 2005, 11:27pm

Hi!

Hab mal nen Patch draus gebaut und FileOps mit

	Source code
1	cReadLine FileOps;

erzeugt ... sollte sonst eigentlich ein Memory-Leak sein oder?

Gruß,
Brougs78

Brougs78 has attached the following file:

tvonscreen-0.7.0-vdr-1.3.18-fix.diff.gz (295 Byte - 588 times downloaded - latest: Mar 31st 2013, 6:02pm)

- -- --- ================================================================ --- -- -
VDR-Konfig: Antec Fusion, Intel E5200, Asus P5N7A-VM (VDPAU), 2xDVB-s2 // yavdr-0.2
- -- --- ================================================================ --- -- -

» Mein VDR-Zeugs (OSDImage-Plugin, Enigma-Skin): brougs78.vdr-developer.org «

Go to the top of the page

BEEjay

Intermediate

Posts: 453

Location: Kirmes am Kanale grande!

Occupation: Programmierer

11

Monday, January 10th 2005, 9:16am

Quoted
Original von Brougs78
Hi!
Hab mal nen Patch draus gebaut und FileOps mit
Source code
1
cReadLine FileOps;
erzeugt ... sollte sonst eigentlich ein Memory-Leak sein oder?

Hi ja, das sollte es sonst sein, ich weis nicht was das bedeutet aber es wurde schon korrigiert von sundin, wie gesagt ist mein erster versuch gewesen und hab auch nicht all zu viel ahnung davon.

@ apex

es sieht so aus als ob irgendwie der font nicht richtig geladen wird oder sowas.
Weil ich hab da auch nichts an der visualisierung groassartig gemacht.

www.sascha-bajonczak.de

Go to the top of the page

_Frank_

Professional

Posts: 597

Location: Miriquidi

Occupation: Mechatroniker

12

Monday, January 10th 2005, 3:39pm

Da ich hier auch das Problem hatte, daß die Anfangszeiten nicht korrekt
angezeigt wurden, hab ich nach Stöbern in der History folgendes gefunden:

- Several formerly non-threadsafe functions now have a return type of cString:
...
cEvent::GetTimeString()
...
Although this will compile without error message, the resulting 'date' will not
be valid after this line. Use this instead:
cString date = DayDateTime();
In a 'const void *' context (as in printf() etc.) the result needs to be
dereferenced with a '*', as in
printf("%s", *DayDateTime());
to make it a 'const char *'.

Hab daraufhin mal, abgeshen vom hier bereits geposteten Patch folgendes in magazine.c geändert:

Zeile 478: const *txt,*timetxt;
in const char *txt;
cString timetxt;

Zeile 545: timetxt=cev->GetTimeString();
in timetxt=*cev->GetTimeString();

kann mir bitte jemand bestätigen, ob dies korrekte Änderungen sind?

Glaub zwar nicht, daß es dabei bleibt, aber zumindest die Anfangszeiten
werden wieder angezeigt.

Go to the top of the page

apex

Intermediate

Posts: 577

13

Monday, January 10th 2005, 5:35pm

@frank

Du bist mein persönlicher Held des Tages

. Mit deinen obigen patches geht TV Onscreen nun wieder!
Dein erstes Posting und dann gleich so ein prima Tip.

This post has been edited 1 times, last edit by "apex" (Jan 10th 2005, 5:37pm)

Go to the top of the page

Brougs78

Master

Posts: 2,450

Location: Tirol (A)

14

Monday, January 10th 2005, 6:02pm

Hi!

Ich denke allerdings, dass da noch einige andere Änderungen folgen müssten, denn wenn man den Sourcecode nach cEvent dursucht gibt es 117 Treffer, wobei sich gerade da einiges geändert hat ... die Frage ist jetzt, ob der Author in den nächsten Tagen ein Update bringt, oder ob man sich da die Arbeit antuen soll das ganze auszubessern ...

Gruß,
Brougs78

- -- --- ================================================================ --- -- -
VDR-Konfig: Antec Fusion, Intel E5200, Asus P5N7A-VM (VDPAU), 2xDVB-s2 // yavdr-0.2
- -- --- ================================================================ --- -- -

» Mein VDR-Zeugs (OSDImage-Plugin, Enigma-Skin): brougs78.vdr-developer.org «

Go to the top of the page

_Frank_

Professional

Posts: 597

Location: Miriquidi

Occupation: Mechatroniker

15

Monday, January 10th 2005, 7:11pm

War mir schon klar, aber ganz so viele Änderungen werden's wohl
nicht werden müssen.

Für das <Suchen> hätte ich jetzt noch was zu ergänzen:

Änderungen in search.c:

Zeile 20: const char *txt,*time1,*time2,*date,*chan;
in const char *txt,*chan;
cString time1,time2,date;

Zeile 30: time1=ev->GetTimeString();
in time1=*ev->GetTimeString();

Zeile 31: time2=ev->GetEndTimeString();
in time2=*ev->GetEndTimeString();

Zeile 33: date=ev->GetDateString();
in date=*ev->GetDateString();

Zeile 48: snprintf(buf,sizeof(buf)-1,"%.6s %s - %s %s/%s",date,time1,time2,chan,txt);
in snprintf(buf,sizeof(buf)-1,"%.6s %s - %s %s/%s",*date,*time1,*time2,chan,txt);

Irgendwann lern ich auch noch, wie man ein File anhängt :-)

Go to the top of the page

AnK

Master

Posts: 2,389

Location: DE

16

Monday, January 10th 2005, 7:14pm

Kann bitte jemand mal ein diff machen und hier hochladen?

HW I: Thermaltake DH 202, Intel DH67BL, Intel Pentium G630, TT Premium S2-6400, Crucial m4 SSD,
Fortron Source Aurum Gold 400W, Scythe Big Shuriken, LG DVD-RW GH22NS50, Kingston ValueRAM 4GB
HW II: A+case Cupid 2, Intel D945GSEJT, 512 MB RAM, 2x PCI Riser, Atric IR-Einschalter Rev. 5,
Hauppauge Nexus Rev 2.2, 60W 12V/DC, Slimline Samsung DVD-RW, CF/SATA-Adapter, 4 GB CF Card