@DEV: Fehler im Sourcecode bouqetlist.cpp

[Massa]

Beim Übersetzen des CDK bin ich gerade mal ein paar der Compiler-Warnings nachgegangen.
Dabei habe ich eindeutige (Tip-)Fehler gefunden:

Sourceode bouqetlist.cpp, ab Zeile 142 (in Funktion int CBouquetList::activateBouquet( int id, bool bShowChannelList)

Code: Alles auswählen

else if ( nNewChannel = -2 )
 {
   // -2 bedeutet EXIT_ALL
  res = menu_return::RETURN_EXIT_ALL;
 }

da steht ein Zuweisungsoperator (=), da muss doch sicher ein == hin, oder?

Das selbe ein paar Zeilen später; Zeile 160 (in Funktion int CBouquetList::exec( bool bShowChannelList)
):

Code: Alles auswählen

	else if ( res = -1)
	{
		// -1 bedeutet nur REPAINT
  

Auch hier sollte es wohl == heissen

[Massa]

Noch so einer in channellist.cpp, Zeile 99 (Funktion int CChannelList::exec() ):

Code: Alles auswählen

else if ( nNewChannel = -1)

[McClean]

sieht auch schlecht aus

[Massa]

Habe gerade ein verlängertes (dbox-und computerfreies) Wochenende hinter mir.
Kome zurück, und sehe im CVS hat sich _JEEEDE_ Menge getan.
Fast alle Warnings, die ich auch schon von Hand bei der von mir ausgecheckten Version beseitigt habe sind weg,
die busybox-Version, auf die ich auch schon upgedatet habe, ist drin...

Jetzt fehlt nur noch der Kernel

Klasse, das ganze! Dickes Lob an die DEVS

P.S.: wenn ich es endlich geschafft habe, meine eigene yadd-Version ohne Fehler vollständig funktionsfähig zu booten, werde ich micvh den DEVS auch anschliessen, und zum reinkommen in die Sourcen mal ein kleines "Codereview" machen
(ich hoffe, das ist erwünscht)

[McClean]

gerne, bist jederzeit willkommen - meld Dich einfach mal, wenn du soweit bist

Gruss McClean

[Massa]

Naja, mein yadd bootet selber und komplett.

Es kommen zwar noch jede Menge Fehlermeldungen, aber immerhin kommt neutrino komplett hoch und läßt sich benutzen.
Ich denke, ich habe noch ein paar falsche Kernel-Konfigurationen und ausserdem auch noch ein paar "spezial Patches" in die busybox eingebaut, die nicht so ganz erfolgreich waren.

Zählt das als erfolgreich ?

Naja, so ganz bin ich nicht zufrieden, aber es dauert immer so lange, das nochmals mit geänderter Konfiguration durchzuziehen...
Und wie schon bemerkt, kann ich eigentlich nur abends testen, und da will meine Frau auch noch Fernseh schauen.
Hmm, irgendwie brauche ich eine zweite dbox zum "spielen"