zapit.patch 26.01.09 #ifndef...

[flasher]

Hi

Gibt es einen Grund warum die beiden Stellen nicht geändert wurden?

Code: Alles auswählen

#ifndef HAVE_DREAMBOX_HARDWARE
	// we need the audiodecoder in standby, to be able to set volume...
	if (audioDecoder) {
		delete audioDecoder;
		audioDecoder = NULL;
	}
#endif

Code: Alles auswählen

#ifndef HAVE_DREAMBOX_HARDWARE
	// needed in standby to correct aspect ration in movieplayer...
	if (videoDecoder) {
		delete videoDecoder;
		videoDecoder = NULL;
	}
#endif

Oder nur vergessen?

Weil wenn: clean up wrong assumptions (undef HAVE_DREAMBOX_HARDWARE is not the same as HAVE_DBOX), wäre es doch logisch das auch zu machen oder etwa nicht?

Gruß

[seife]

Einfach nur vergessen.

[flasher]

Gut!
Hätte ja auch einen anderen Grund haben können.

[seife]

Tatsächlich ist es nicht vergessen, sondern korrekt.
Auf der dreambox brauche ich den Audio- und Videodekoder auch im Standby, um Lautstärke und AR-Korrekturen zu machen, wenn z.B. der movieplayer läuft (wie es im SCARTMODE auf den grösseren dreamboxen ist weiss ich nicht, da ich keine mit SCART-Eingang habe.

Bei anderen boxen (Tripledragon, anyone?) kann das wieder anders sein, insofern ist das #ifdef so schon korrekt.

[flasher]

Auf der dreambox brauche ich den Audio- und Videodekoder auch im Standby, um Lautstärke und AR-Korrekturen zu machen, wenn z.B. der movieplayer läuft (wie es im SCARTMODE auf den grösseren dreamboxen ist weiss ich nicht, da ich keine mit SCART-Eingang habe.

Ähm jetzt verstehe ich gar nichts mehr.
Wenn Du Audio/Video auf der Dream brauchst, dann ist doch #ifndef HAVE_DREAMBOX_HARDWARE falsch.
Damit hättest Du das auf der Dream doch garnicht weil HAVE_DREAMBOX_HARDWARE doch defined wäre.

Und wenn wir das auf der dbox brauchen und auf der Dream sowieso (zumindest die Kleine), dann könnte man das #if... doch auch gleich weg lassen.
Oder gibts für die kleine Dream ein eigenes alá HAVE_DREAM_500 ???

Verwirrung

[seife]

Schau mal was in dem #ifdef gemacht wird, indem Code den du zitiert hast. Ich glaube, das ist dei standby()-Funktion vom zapit, und da wird normalerweise der audio und videodekoder delete[]d. Auf der dreambox brauche ich aber woanders audioDecoder->setVolume() auch im standby, also darf ich den nicht deleten.

...übrigens auch ein Grund, controld und zapit zusammenzulegen ;-)

[flasher]

Hi

Du schreibst immer wieder #ifdef, dort steht aber #ifndef...
Demnach sollte die Funktion auf Dreamboxen doch garnicht zur Verfügung stehen. Das allein ist das was mich gerade verwirrt!!!
Es sein denn, wie schon einmal geschrieben, es gibt sowas wie #define DREAMBOX_500 um die kleine Dream zu identifizieren, dann
würde ich das #ifndef ja auch, im Bezug auf Deine Ausführung von "kleine" Dreambox, verstehen.

[seife]

Normalerweise wird im standby der (video|audio)Decoder delete[]d.
Auf der dreambox brauche ich den aber, für aspect und volume

also:

Code: Alles auswählen

#ifndef DREAM
        delete[]
#elsif

Natürlich könnte man auch "#ifdef DBOX" machen, aber das Fasse ich dann an, wenn ich mindestens eine weitere Plattform habe, auf der man einen Unterschied machen muss.

Das funktioniert so, das war ja auch Jahrelang jetzt so richtig und ich habe das sowohl mit als auch ohne -DHAVE_DREAMBOX_HARDWARE probiert

[flasher]

Hi

Ich habe mir den Thread jetzt 8x durchgelesen und erst jetzt meinen Denkfehler bemerkt.
Ich geh dann mal eben in den Keller Kohle putzen bis die weiß ist!!!

Gruß

[Houdini]

Ich geh dann mal eben in den Keller Kohle putzen bis die weiß ist!!!

Den kannte ich auch noch nicht..