[ANNOUNCE] MiniPatch 0.0.5

Hallo Marcus,

habe mir das diff angesehen und hätte es gerne.
Für die 1.2.6 wird's das wohl nicht mehr geben, oder?

lg, Gerhard

Hi,

supi das es das für die 1.3.6 gibt. Aber ich habs gerade mal probiert, mit nem frischen vdr-1.3.6 + Elchii 4d, ich bekomme einen reject in der vdr.c, der aber manuell geändert werden kann.

Beim compilieren kommt aber dann:

g++ -g -O2 -Wall -Woverloaded-virtual -c -DREMOTE_KBD -DREMOTE_ -D_GNU_SOURCE -DVIDEODIR=\"/video\" -DPLUGINDIR=\"./PLUGINS/lib\" -I../DVB/include timers.c
timers.c:51: error: default argument given for parameter 2 of `
cTimer::cTimer(const cEvent*, bool = false)'
timers.h:46: error: after previous specification in `cTimer::cTimer(const
cEvent*, bool = false)'
timers.c:509: error: default argument given for parameter 1 of `cTimer*
cTimers::GetNextActiveTimer(bool = false)'
timers.h:102: error: after previous specification in `cTimer*
cTimers::GetNextActiveTimer(bool = false)'
make: *** [timers.o] Error 1

Fehler in timers.c...

Grüsse
AB

Nochmal ich,

ja das ist erstmal ok, sollte klappen nun bekomm ich aber noch die Meldung:

g++ -g -O2 -Wall -Woverloaded-virtual -c -DREMOTE_KBD -DREMOTE_ -D_GNU_SOURCE -DVIDEODIR=\"/video\" -DPLUGINDIR=\"./PLUGINS/lib\" -I../DVB/include svdrp.c
svdrp.c: In member function `void cSVDRP::CmdNEXT(const char*)':
svdrp.c:992: error: no matching function for call to `cTimers::
GetNextActiveTimer()'
timers.h:102: error: candidates are: cTimer* cTimers::GetNextActiveTimer(bool)
make: *** [svdrp.o] Error 1

Die entsprechende Funktion verlangt noch nach nem bool Parameter.

AB

hi,

thx, hatte es jetzt probehalber auf true geändert, da hatte es auch kompiliert, aber du wirst dich ja mit den Funktionen auskennen ich werds dann mal wieder in false ändern...

Danke nochmal

AB

Hallo Mini,

Zuerst lob für die Gute arbeit

Zitat

Original von mini
-Introduced a separate boolean SwitchTimer field for
timers and now drawing two icons in the timer list

kannst du mir verraten welchen bit des Timer ID's du dafür nutzt ?

Kann man das evt. recht einfach ändern ? (master-timer kollidiert damit - kann aber auch sein das es in eine änderung von selbigen endet ;))

Zitat

-Added support for prefered languages for DVB audio
tracks and an option to autoswitch to the second
audio channel when starting to replay a recording

ich wies nicht wie einfach das geht aber man könnte auch statdessen die reihenfolge der Audio PIDs ändern. Hätte natürlich den nachteil das es mit alten aufnahmen nicht funktioniert.

Ich nutze schon lange VDR und habe einige probleme mit der änderung der gelb/grün/blaue tasten während der kanal-umschalt-OSD anzeige. Auf der anderen seite möchte ich aber unter anderen nicht auf die Anzeige der Timer und Aufnahmen im OSD verzichten
Wäre es möglich eine Option zu erstellen der wahlweise das alte verhalten wiederherstellt ?

Wäre echt klasse

Gruß
Viking

Hi!
Nice patch you have made, i was really looking forward to the audio language part, but it didnt work very well for channels with >2 audio pids. So i made some code for putting my preffered language first in the apids array. This code should go in pat.c, i dont know if this is the best way of doing this but i couldnt see any easyer way, but i hope you could fix this problem.
Here is what i did, feel free to use this in you patch:

if (Setup.UpdateChannels >= 2) {
//*****my code for the preffered audio lang as nr 1 in apids array*****
//This i based on the minipatch,
//but the minipatch didnt work for me so i had to do this.
//this works pretty well, but as this is my first go at c++ and vdr
//i would really like to get comments on this...
// this code goes in pat.c
bool found = false;
int tempApid = 0;
int currentIdx = 0;
for (int k = 0;k<3 && Setup.DVBLanguages[k]>0;k++){
for(int j = 0;j<NumApids;j++){
if(Setup.DVBLanguages[k]==I18nLanguageIndex(ALangs[j])){
tempApid = Apids[currentIdx];
Apids[currentIdx] = Apids[j];
Apids[j] = tempApid;
strn0cpy(ALangs[j],ALangs[currentIdx], 4);
strn0cpy(ALangs[0],I18nLanguageCode(Setup.DVBLanguages[k]), 4);
found = true;
break;
//currentIdx++;
}//if
}//for
if (found) break;
}//for
//******end of my code*******
Channel->SetPids(Vpid, Vpid ? Ppid : 0, Apids, ALangs, Dpids, DLangs, Tpid);
Channel->SetCaIds(CaDescriptors->CaIds());
}//if

I have got a problem with that patch applied on VDR 1.3.6 : when I launch the games plugin I have a crash !!

If you want I made my own patch based on minipatch : http://trois.six.free.fr/fichiers/vdr-1.3.6-36.diff.bz2

It contains :

Zitat

ElchiAIO 4d
Minipatch 0.0.5
Patch libsi
Patch settime-fix
lnbsharing
jumpingseconds
jump
AC3OverDVB
Subtitles & Ttxtsubs
OSDTeletext reinsert-teletext
Icon modified
DisableDoubleEpgEntrys 0.6

Alles anzeigen

but I do have a problem with the games plugin...

Hallo Mini,

mit der Version für die 1.2.6 wird's wahrscheinlich ja nix mehr, oder?

Ich hätte dann noch einen Vorschlag für den nächsten MiniPatch.

Wie wäre es, wenn man auch im EPG-Viewer diese Punkte anzeigen würden, wenn es Beschreibung für den EPG-Eintrag gibt?
Momentan sieht man erst wenn man ok drückt, ob es eine Beschreibung gibt oder nicht.

lg, Gerhard

Hallo Minni,

bin mittlerweile auf die 1.3.20 umgestiegen.
Kann man Deinen Patch hier auch einfach anwenden?

lg, Gerhard

Hi mini,

die Farbstasten (grün und gelb) waren doch nur aktiv, wenn die Programminfo's angezeigt wurden - richtig ?

Kann das nicht trotzdem eingebaut werden ? Ist doch in erster Linie unwichtig, ob die Quadrate angezeigt werden. Das kann man dann später in die Skins einbauen ... ( if Patch then Quadrat -> oder so )

Gruß
Michael

Zitat

Original von mini
Ich habe jetzt auf 1.3.21 upgedated und zumindest die switchtimer-Funktion wieder eingepatcht. Ausserdem ändert der Patch die mögliche Zeit zwischen zwei Zahleneingaben bei der direkten Programmanwahl auf 2 Sekunden und eine einmal mit OK manuell eingeblendete Progarmminfo verschwindet nicht mehr automatisch, sondern muss mit OK wieder "ausgeschaltet" werden

Fav-Audio ist ja mittlerweile in den vdr selbst übernommen worden, ist also nicht mehr enthalten. Für die alte Hauptfunktion des Patches, also Einblendung der nächsten *4* Sendungsinfos und direkte Anzeige der Programminfo über die Farbtasten sehe ich momentan keine Möglichkeit.

Marcus

hoihoi und hallo.

ich wäre hellauf begeistert, wenn du deinen patch auch für 1.3.24 mal checken würdest.... ich bekomme da nee Menge an Hunks und Rejects -

danke im vorraus. mfg bn

Hallo,

ich habe mich mal mit dem Minipatch beschäftigt, und es scheint nun auch mit VDR 1.3.24 zu funktionieren. Bitte mit einem Vanilla VDR intensiv testen UND berichten!!!

Falls es nicht klappt, sorry ich kann kein C geschweige denn C++ und fürchte das müsste sich dann ein richtiger Developer dann anschauen.
Ich entschuldige mich schon mal für den schäuslich geänderten Code.

Tschau,
Frank

Moin,

upps im Patch von gestern war zusätzlich noch der missing Patch enthalten.

Tschau,
Frank