Nachdem ich (vor-)gestern einen Patch fürs Scan-Setup erstellt habe und mir jetzt das ganze nochmal durch den Kopf lassen ging, bin ich der Meinung, dass der Code in settings.cpp von Zeile 349 bis 373 völlig überflüssig ist. Genau derselbe Code wird ja in scan_setup.cpp von Zeile 394 bis 420 aufgerufen und zwar jedes Mal, wenn man das Scan-Setup öffnet. Ich denke der Code in settings.cpp ist noch ein Überbleibsel aus der Zeit, als die Menüs nur einmal beim Start von Neutrino aufgebaut wurden und dann im Speicher blieben. Ausprobieren kann ich es nicht, da ich keine Sat-Box habe. Ich kann den Code aber mal umbauen und ihr könnt mir sagen, ob das Kanalsuchemenü noch funktioniert.
Der Patch dafür ist folgender. Ich habe den mal schnell zusammengebaut, aber nicht getestet.
Scan setup: fix memleak and remove unnecessary code
Link zum Patch entfernt
Edit: Ich habe noch eine Kleinigkeit hinzugefügt. In settings.h wird zusätzlich noch ein überflüssig gewordenes Include entfernt.
Edit: Mist, durch das Entfernen der Include-Datei menue.h kompiliert das nicht mehr, weil er neutrino_msg_t dann nicht mehr kennt. Also packen wir eben rcinput.h mit rein. Da wird neutrino_msg_t schließlich definiert. Kompilieren kann man es nun also schon mal.
Überflüssiger Code in settings.cpp
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Überflüssiger Code in settings.cpp
Ich habe den Code nochmals überarbeitet. So gefällt mir das besser.
Scan setup: fix memleak and remove unnecessary code
Link entfernt, da Patch im CVS
Scan setup: fix memleak and remove unnecessary code
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
-
- Administrator
- Beiträge: 2675
- Registriert: Donnerstag 28. September 2006, 19:18
Re: Überflüssiger Code in settings.cpp
Den Scan braucht man ja nicht so oft, also etwas Geduld ;-)
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Überflüssiger Code in settings.cpp
ditoGaucho316 hat geschrieben:Ausprobieren kann ich es nicht, da ich keine Sat-Box habe.
-
- Image-Team
- Beiträge: 1015
- Registriert: Freitag 7. Februar 2003, 18:37
Re: Überflüssiger Code in settings.cpp
Ich habs hier mal getestet, gebaut hat es.
Senderscan auf Astra geht - danach reboot - geht auch dann noch alles Menu sieht okay aus.
Dann auf Diseq 1.0 eingestellt und Asta/Hotbird gescannt - geht auch
Im seriellen Log waren keine Fehler zu sehen.
Gruß Riker
Senderscan auf Astra geht - danach reboot - geht auch dann noch alles Menu sieht okay aus.
Dann auf Diseq 1.0 eingestellt und Asta/Hotbird gescannt - geht auch
Im seriellen Log waren keine Fehler zu sehen.
Gruß Riker
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Überflüssiger Code in settings.cpp
committed:Gaucho316 hat geschrieben:scan-setup_fix-memleak-and-remove-unnecessary-code_2012-06-27_2040.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3518
http://article.gmane.org/gmane.comp.vid ... x.scm/3519