Commit-Wünsche für das CVS II
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Es geht nicht um die Aufnahme-Einstellungen sondern um die Movieplayer-Einstellungen.
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
In die 2. Variante des Patches hat sich leider ein grober Fehler eingeschlichen. Hat man den Streamingserver deaktiviert, wurde auch das neue Streamingserver-Einstellungsmenü deaktiviert und man konnte ihn nie wieder aktivieren. Das war noch ein Überbleibsel aus der ersten Variante.Gaucho316 hat geschrieben:Dann habe ich noch einen Patch für die Movieplayer-Einstellungen. Ich finde das Menü ziemlich überfrachtet. Deshalb habe ich die Streamingserver-Einstellungen in ein Untermenü verfrachtet.
...
Ich habe hier noch eine andere Variante, die mir persönlich besser gefällt. Sucht euch eine aus.
Movieplayer setup: add submenu for streaming server settings
movieplayer-setup_add-submenu-for-streaming-server-settings_2012-09-02_1719.diff
Movieplayer setup: add submenu for streaming server settings
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Durch den vorherigen Patch ist mir aufgefallen, dass das Deaktivieren des Streamingservers (VLC) in den Movieplayer-Einstellungen überhaupt keine Auswirkungen hat. Es werden nur die Menüpunkte im Einstellungsmenü deaktiviert. Sie bleiben aber aktiv. Deshalb habe ich eine Hinweisbox eingebaut, die beim Aufruf der Menüpunkte "Datei via VLC", "DVD" und "(S)VCD" erscheint, wenn der Streamingserver deaktiviert ist. Anschließend wird der Movieplayer wieder beendet.
Movieplayer: show hint if streaming server disabled
Link zum Patch entfernt
Edit: Ich habe da noch eine andere Idee im Kopf. Man könnte die Menüpunkte, mit denen man den Movieplayer im VLC-Modus aufruft, auch deaktivieren, wenn man den Streamingserver deaktiviert hat. Ich muss aber erst einmal sehen, ob das überhaupt so funktioniert, wie ich mir das denke.
Edit: Ich habe meine Idee nun mal umgesetzt. Funktionieren tut's. Man kann sich dann den vorherigen Patch sparen.
Movieplayer menu: disable streaming server entries if streaming server disabled
Link entfernt, da Patch im CVS
Movieplayer: show hint if streaming server disabled
Link zum Patch entfernt
Edit: Ich habe da noch eine andere Idee im Kopf. Man könnte die Menüpunkte, mit denen man den Movieplayer im VLC-Modus aufruft, auch deaktivieren, wenn man den Streamingserver deaktiviert hat. Ich muss aber erst einmal sehen, ob das überhaupt so funktioniert, wie ich mir das denke.
Edit: Ich habe meine Idee nun mal umgesetzt. Funktionieren tut's. Man kann sich dann den vorherigen Patch sparen.
Movieplayer menu: disable streaming server entries if streaming server disabled
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Wenn ihr den vorherigen Patch für das Movieplayer-Menü mal getestet habt, könnte euch aufgefallen sein, dass beim Deaktivieren und Aktivieren des Streamingservers über das Einstellungsmenü im Movieplayer-Menü ganz kurz die VLC-Menüpunkte auftauchen und dann aber sofort wieder übermalt werden. Da das nicht besonders schön ist, habe ich folgenden Patch erstellt.
Er löst auch gleichzeitig ein Problem im Moviebrowser, das ich entdeckt habe. Wählt man einen der unteren Einträge in Einstellungen -> Film Informationen -> Markierungen und ändert den Namen, wird fälschlicherweise ein Menüeintrag am unteren Bildschirmrand gezeichnet.
CMenuWidget: don't paint hidden menu items on setActive changes
Link entfernt, da Patch im CVS
Er löst auch gleichzeitig ein Problem im Moviebrowser, das ich entdeckt habe. Wählt man einen der unteren Einträge in Einstellungen -> Film Informationen -> Markierungen und ändert den Namen, wird fälschlicherweise ein Menüeintrag am unteren Bildschirmrand gezeichnet.
CMenuWidget: don't paint hidden menu items on setActive changes
Link entfernt, da Patch im CVS
Re: Commit-Wünsche für das CVS II
Hi,
die "neuen" Patches von @Gaucho316 (Streamingserver Variante 2 und folgende) laufen bei mir ohne Auffälligkeiten.
Ich lass es aber noch von "Anderen" testen.
Aber im Prinzip ... man könnte es einchecken ...
MfG
Edit:
Auch das Einrücken von Untermenüs zu den jeweiligen Überschriften ist "sauber" ...
die "neuen" Patches von @Gaucho316 (Streamingserver Variante 2 und folgende) laufen bei mir ohne Auffälligkeiten.
Ich lass es aber noch von "Anderen" testen.
Aber im Prinzip ... man könnte es einchecken ...
MfG
Edit:
Auch das Einrücken von Untermenüs zu den jeweiligen Überschriften ist "sauber" ...
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3568
http://article.gmane.org/gmane.comp.vid ... x.scm/3570
http://article.gmane.org/gmane.comp.vid ... x.scm/3571
PS:
http://article.gmane.org/gmane.comp.vid ... x.scm/3567Gaucho316 hat geschrieben:neutrino_use-addintroitems-in-menus_2012-09-01_2250.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3568
http://article.gmane.org/gmane.comp.vid ... x.scm/3569Gaucho316 hat geschrieben:movieplayer-setup_add-submenu-for-streaming-server-settings_2012-09-04_1959.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3570
http://article.gmane.org/gmane.comp.vid ... x.scm/3571
http://article.gmane.org/gmane.comp.vid ... x.scm/3572Gaucho316 hat geschrieben:movieplayer-menu_disable-streaming-server-entries-if-streaming-server-disabled_2012-09-05_1920.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3573Gaucho316 hat geschrieben:menue_dont-paint-hidden-menu-items-on-setactive-changes_2012-09-06_1842.diff
PS:
http://article.gmane.org/gmane.comp.vid ... x.scm/3577Gaucho316 hat geschrieben:menue_fix-icon-offset-calculation_2012-08-29_1957.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Und weiter geht's.
In einigen Menüklassen lassen sich die Konstruktoren vereinfachen, wenn man die Standardwerte sinnvoller setzt.
Neutrino: simplify some menu class constructors
Link entfernt, da Patch im CVS
Dann habe ich einige fehlende Menü-Untertitel ergänzt, damit es einheitlich ist:
Einstellungen -> OSD -> Sprache
Einstellungen -> OSD -> Farben -> Theme
Service -> Software-Aktualisierung
Service -> Software-Aktualisierung -> *
Und im Moviebrowser habe ich das komplette Einstellungemenü mit Untertiteln versehen.
Neutrino: add subheads to some menus
Link entfernt, da Patch im CVS
Dabei ist mir aufgefallen, dass der Hintergrund von Menü-Untertiteln mit zu langen Texten über das Fenster hinaus gezeichnet wird.
Menu separator: fix subhead painting
Link entfernt, da Patch im CVS
Außerdem habe ich im Menü "Internetradios einlesen" des Audioplayers, in der EPG-Suche und in der Timerliste beim Anlegen und Bearbeiten eines Timers den Zurück-Button gegen den Abbrechen-Button getauscht. Der passt meiner Meinung nach dort besser.
Neutrino: use cancel button in some menus
Link entfernt, da Patch im CVS
Schließlich habe ich die fehlenden "Intro Items" im Mountchooser und im EPG Plus in den Menüs, die man über die rote und blaue Taste erreicht, ergänzt.
Neutrino: add intro items to some menus
Link entfernt, da Patch im CVS
Edit: Ich habe den Patch "Menu separator: fix subhead painting" noch ein klein wenig optimiert.
Edit: Ich habe noch eine Stelle gefunden, an der Menü-Untertitel fehlen, und deshalb den Patch "Neutrino: add subheads to some menus" auch noch einmal ergänzt.
In einigen Menüklassen lassen sich die Konstruktoren vereinfachen, wenn man die Standardwerte sinnvoller setzt.
Neutrino: simplify some menu class constructors
Link entfernt, da Patch im CVS
Dann habe ich einige fehlende Menü-Untertitel ergänzt, damit es einheitlich ist:
Einstellungen -> OSD -> Sprache
Einstellungen -> OSD -> Farben -> Theme
Service -> Software-Aktualisierung
Service -> Software-Aktualisierung -> *
Und im Moviebrowser habe ich das komplette Einstellungemenü mit Untertiteln versehen.
Neutrino: add subheads to some menus
Link entfernt, da Patch im CVS
Dabei ist mir aufgefallen, dass der Hintergrund von Menü-Untertiteln mit zu langen Texten über das Fenster hinaus gezeichnet wird.
Menu separator: fix subhead painting
Link entfernt, da Patch im CVS
Außerdem habe ich im Menü "Internetradios einlesen" des Audioplayers, in der EPG-Suche und in der Timerliste beim Anlegen und Bearbeiten eines Timers den Zurück-Button gegen den Abbrechen-Button getauscht. Der passt meiner Meinung nach dort besser.
Neutrino: use cancel button in some menus
Link entfernt, da Patch im CVS
Schließlich habe ich die fehlenden "Intro Items" im Mountchooser und im EPG Plus in den Menüs, die man über die rote und blaue Taste erreicht, ergänzt.
Neutrino: add intro items to some menus
Link entfernt, da Patch im CVS
Edit: Ich habe den Patch "Menu separator: fix subhead painting" noch ein klein wenig optimiert.
Edit: Ich habe noch eine Stelle gefunden, an der Menü-Untertitel fehlen, und deshalb den Patch "Neutrino: add subheads to some menus" auch noch einmal ergänzt.
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Den ersten Patch habe ich mal übernommen und noch ein klein wenig erweitert, um zu verhindern, dass Neutrino nie herunterfährt, wenn beim Speichern der EPG-Daten ein Fehler auftritt. Ob der zweite Patch auch wichtig ist, kann ich nicht einschätzen.GetAway hat geschrieben:Satbaby hat einen weiteren möglichen Segfault in sectionsd.cpp gefixt.
http://git.coolstreamtech.de/?p=cst-pub ... 0a9704b3a6
Eventuell wäre dieser Fix auch noch etwas.
http://git.coolstreamtech.de/?p=cst-pub ... cb6b68d03b
sectionsd: fix possible segfault and never ending shutdown
Link entfernt, da Patch im CVS
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3592
http://article.gmane.org/gmane.comp.vid ... x.scm/3587Gaucho316 hat geschrieben:neutrino_simplify-some-menu-class-constructors_2012-09-14_1918.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3588Gaucho316 hat geschrieben:neutrino_add-subheads-to-some-menus_2012-09-15_2145.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3589Gaucho316 hat geschrieben:menue_fix-subhead-painting_2012-09-15_2029.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3590Gaucho316 hat geschrieben:neutrino_use-cancel-button-in-some-menus_2012-09-15_1750.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3591Gaucho316 hat geschrieben:neutrino_add-intro-items-to-some-menus_2012-09-15_1817.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3592
http://article.gmane.org/gmane.comp.vid ... x.scm/3593Gaucho316 hat geschrieben:sectionsd_fix-possible-segfault-and-never-ending-shutdown_2012-09-16_2011.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Im Coolstream GIT wurde ein Schreibfehler in der XML-Datei mit den Aufnahme-Informationen korrigiert. Ich habe das mal übernommen, gebe aber zu Bedenken, dass dadurch der Eintrag <qualitiy>...</qualitiy> in vorhandenen XML-Dateien nicht mehr eingelesen wird. Da man diesen Wert aber so oder so von Hand eingeben muss, finde ich das nicht schlimm.
Movieinfo: fix typo in xml-entry "quality"
Link zum Patch entfernt
Movieinfo: fix typo in xml-entry "quality"
Link zum Patch entfernt
-
- Developer
- Beiträge: 4189
- Registriert: Sonntag 2. November 2003, 12:36
Re: Commit-Wünsche für das CVS II
Damit macht man aber die Arbeit zunichte, wenn das jemand von hand eingegeben hat.
Aus Kompatibilitätsgründen sollte man also beide Varianten einlesen, aber nur noch die richtige rausschreiben.
...das ist so ein Beispiel einer Änderung, wegen der dann alle immer sagen "Settingsübernahme ist böse", obwohl man bei defensiver Programmierung damit keine Probleme hat
Aus Kompatibilitätsgründen sollte man also beide Varianten einlesen, aber nur noch die richtige rausschreiben.
...das ist so ein Beispiel einer Änderung, wegen der dann alle immer sagen "Settingsübernahme ist böse", obwohl man bei defensiver Programmierung damit keine Probleme hat
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Ok, hast mich überzeugt. Der zusätzliche Code ist ja auch recht klein.
Movieinfo: fix typo in xml-entry "quality"
Link entfernt, da Patch im CVS
Edit: Ich habe vorsichtshalber die Einlesereihenfolge von <quality>...</quality> und <qualitiy>...</qualitiy> getauscht.
Movieinfo: fix typo in xml-entry "quality"
Link entfernt, da Patch im CVS
Edit: Ich habe vorsichtshalber die Einlesereihenfolge von <quality>...</quality> und <qualitiy>...</qualitiy> getauscht.
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3596Gaucho316 hat geschrieben:movieinfo_fix-typo-in-xml-entry-quality_2012-09-24_1959.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Heute ist mir mal wieder aufgefallen, dass im Movieinfo-Fenster (erreichbar über die ?-Taste im Movieplayer und Moviebrowser) der Dateiname oft abgeschnitten dargestellt wird. Der folgende Patch hilft dagegen, indem nun auch ein automatischer Umbruch nach Slashes und Unterstrichen möglich ist.
CTextBox: break lines also after slashes and underscores
Link entfernt, da Patch im CVS
Dabei habe ich bemerkt, dass in der Klasse CTextBox die XML-Sonderzeichen unnötigerweise dekodiert werden. Meiner Meinung nach können die hier gar nicht mehr vorkommen, weil die Sonderzeichen schon vorher an anderer Stelle dekodiert werden.
CTextBox: remove unnecessary special char decoding
Link entfernt, da Patch im CVS
Und weil ich schon mal dabei bin, habe ich außerdem noch zwei Patches für die Klasse CTextBox aus dem Coolstream GIT übernommen.
CTextBox: prevent possible division by zero
Link entfernt, da Patch im CVS
CTextBox: fix string access outside of its size
Link entfernt, da Patch im CVS
CTextBox: break lines also after slashes and underscores
Link entfernt, da Patch im CVS
Dabei habe ich bemerkt, dass in der Klasse CTextBox die XML-Sonderzeichen unnötigerweise dekodiert werden. Meiner Meinung nach können die hier gar nicht mehr vorkommen, weil die Sonderzeichen schon vorher an anderer Stelle dekodiert werden.
CTextBox: remove unnecessary special char decoding
Link entfernt, da Patch im CVS
Und weil ich schon mal dabei bin, habe ich außerdem noch zwei Patches für die Klasse CTextBox aus dem Coolstream GIT übernommen.
CTextBox: prevent possible division by zero
Link entfernt, da Patch im CVS
CTextBox: fix string access outside of its size
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Eine andere Sache, die mir immer mal wieder auffällt, ist, dass im Fenster mit den Sendungsinformationen (zweimal die ?-Taste im TV-Modus drücken) manchmal der Info1-Text (der kursiv geschriebene Text) fehlt. Das kommt dadurch, dass im Info2-Text nach dem Info1-Text gesucht wird. Wird der Info1-Text gefunden, wird er nicht angezeigt. Das habe ich diese Woche wieder bei der Serie "Alcatraz" bemerkt, weil dort die Folgentitel nach den gesuchten Personen benannt sind und diese auch im Info2-Text vorkommen. Auf den Sky Filmkanälen, bei denen im Info1-Text das Genre gesendet wird, kann man das auch gut beobachten. Ich finde dieses Verhalten nicht besonders schön. Deshalb habe ich den entsprechenden Code so abgeändert, dass der Info1-Text nur dann nicht mehr angezeigt wird, wenn er am Anfang des Info2-Textes gefunden wird. So wird es ein paar Zeilen höher im Code schließlich auch gemacht.
EPG Data Viewer: only hide info1 if info2 begins with info1
Link entfernt, da Patch im CVS
EPG Data Viewer: only hide info1 if info2 begins with info1
Link entfernt, da Patch im CVS
Re: Commit-Wünsche für das CVS II
Hi,Gaucho316 hat geschrieben:Eine andere Sache, die mir immer mal wieder auffällt, ist, dass im Fenster mit den Sendungsinformationen (zweimal die ?-Taste im TV-Modus drücken) manchmal der Info1-Text (der kursiv geschriebene Text) fehlt. Das kommt dadurch, dass im Info2-Text nach dem Info1-Text gesucht wird. Wird der Info1-Text gefunden, wird er nicht angezeigt. Das habe ich diese Woche wieder bei der Serie "Alcatraz" bemerkt, weil dort die Folgentitel nach den gesuchten Personen benannt sind und diese auch im Info2-Text vorkommen. Auf den Sky Filmkanälen, bei denen im Info1-Text das Genre gesendet wird, kann man das auch gut beobachten. Ich finde dieses Verhalten nicht besonders schön. Deshalb habe ich den entsprechenden Code so abgeändert, dass der Info1-Text nur dann nicht mehr angezeigt wird, wenn er am Anfang des Info2-Textes gefunden wird. So wird es ein paar Zeilen höher im Code schließlich auch gemacht.
EPG Data Viewer: only hide info1 if info2 begins with info1
epgview_only-hide-info1-if-info2-begins-with-info1_2012-10-04_1903.diff
funktioniert ...
Werde mir unverschämterweise herausnehmen, dieses auch in meinem Image zu benutzen ...
MfG
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Im Movieinfo-Fenster fehlt die Anzeige des Genres. Ich habe den entsprechenden Code einfach mal aus dem EPG-Viewer übernommen. Vielleicht gibt es ja noch eine einfachere Möglichkeit, aber wenn niemandem etwas besseres einfällt, könnte meine Variante ins CVS.
Movieinfo: show genre
Link zum Patch entfernt
Movieinfo: show genre
Link zum Patch entfernt
-
- Contributor
- Beiträge: 1509
- Registriert: Donnerstag 27. Dezember 2007, 12:59
Re: Commit-Wünsche für das CVS II
nfs.cpp: disable mount option if mountpoint is mounted. Ported from N-HD
disable mount option if mountpoint is mounted_2012_10_11_1818.diff
disable mount option if mountpoint is mounted_2012_10_11_1818.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Ich habe jetzt doch noch eine andere Variante des Patches, bei der das Kopieren der Locale-Arrays nach movieinfo.cpp entfällt. Mir gefällt diese Version besser.Gaucho316 hat geschrieben:Im Movieinfo-Fenster fehlt die Anzeige des Genres. Ich habe den entsprechenden Code einfach mal aus dem EPG-Viewer übernommen. Vielleicht gibt es ja noch eine einfachere Möglichkeit, aber wenn niemandem etwas besseres einfällt, könnte meine Variante ins CVS.
Movieinfo: show genre
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1509
- Registriert: Donnerstag 27. Dezember 2007, 12:59
Re: Commit-Wünsche für das CVS II
Tuxboxapi: tell sectionsd after registering neutrino, where we are.
TODO: find a way to do it similar with sectionsdcontrol
tell_sectionsd_where_we_are!_tuxboxapi_2012_10_15_2243.diff
TODO: find a way to do it similar with sectionsdcontrol
tell_sectionsd_where_we_are!_tuxboxapi_2012_10_15_2243.diff
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
commited:
http://article.gmane.org/gmane.comp.vid ... x.scm/3602
http://article.gmane.org/gmane.comp.vid ... x.scm/3597Gaucho316 hat geschrieben:textbox_break-lines-also-after-slashes-and-underscores_2012-10-02_1934.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3598Gaucho316 hat geschrieben:textbox_remove-unnecessary-special-char-decoding_2012-10-02_2002.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3599Gaucho316 hat geschrieben:textbox_prevent-possible-division-by-zero_2012-10-02_1932.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3600Gaucho316 hat geschrieben:textbox_fix-string-access-outside-of-its-size_2012-10-02_1933.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3601Gaucho316 hat geschrieben:movieinfo_show-genre_2012-10-11_1922.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3602
http://article.gmane.org/gmane.comp.vid ... x.scm/3603Gaucho316 hat geschrieben:epgview_only-hide-info1-if-info2-begins-with-info1_2012-10-04_1903.diff
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3604GetAway hat geschrieben:disable mount option if mountpoint is mounted_2012_10_11_1818.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3605GetAway hat geschrieben:tell_sectionsd_where_we_are!_tuxboxapi_2012_10_15_2243.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Der folgende Patch schließt drei Speicherlecks in Neutrino im Zusammenhang mit delete. Ich habe ihn aus dem Coolstream GIT übernommen.
Neutrino: fix memleaks
Link entfernt, da Patch in Git
Die Idee für den nächsten Patch stammt auch aus dem Coolstream GIT. Ich habe es aber codetechnisch etwas anders gelöst. Hat man einen Sleeptimer gesetzt, wird man eine Minute vor Ausführung nicht mehr nur wie bisher gewarnt, sondern man hat die Möglichkeit, den Vorgang abzubrechen. Das Fenster wird für 30 Sekunden eingeblendet. Zusätzlich habe ich das Abbruch-Fenster bei Shutdowntimern auch so umgestellt, dass es für 30 Sekunden angezeigt wird. Bisher war das dort auf 5 Sekunden eingestellt, hat aber gar nicht funktioniert, weil ein Funktionsparameter gefehlt hat, so dass das Fenster nie ausgeblendet wurde. 5 Sekunden waren mir übrigens etwas zu kurz.
Neutrino: add possibility to cancel sleep timer
Link entfernt, da Patch im CVS
Neutrino: fix memleaks
Link entfernt, da Patch in Git
Die Idee für den nächsten Patch stammt auch aus dem Coolstream GIT. Ich habe es aber codetechnisch etwas anders gelöst. Hat man einen Sleeptimer gesetzt, wird man eine Minute vor Ausführung nicht mehr nur wie bisher gewarnt, sondern man hat die Möglichkeit, den Vorgang abzubrechen. Das Fenster wird für 30 Sekunden eingeblendet. Zusätzlich habe ich das Abbruch-Fenster bei Shutdowntimern auch so umgestellt, dass es für 30 Sekunden angezeigt wird. Bisher war das dort auf 5 Sekunden eingestellt, hat aber gar nicht funktioniert, weil ein Funktionsparameter gefehlt hat, so dass das Fenster nie ausgeblendet wurde. 5 Sekunden waren mir übrigens etwas zu kurz.
Neutrino: add possibility to cancel sleep timer
Link entfernt, da Patch im CVS
-
- Developer
- Beiträge: 4189
- Registriert: Sonntag 2. November 2003, 12:36
Re: Commit-Wünsche für das CVS II
Das muss nicht richtig sein, weil der rcinput-Code und der infoviewer-Code (der da leider auch mit involviert ist...) unterschiedlich sind. Beim neutrino habe ich das vor langem schon mal gefixt, im neutrino-HD ist das immer noch kaputt.Gaucho316 hat geschrieben:Der folgende Patch schließt drei Speicherlecks in Neutrino im Zusammenhang mit delete. Ich habe ihn aus dem Coolstream GIT übernommen.
Neutrino: fix memleaks
neutrino_fix-memleaks_2012-10-19_2056.diff
Du musst wirklich genau schauen wo die daten mit welchem Typ alloziert wurden um dann zu wissen, wie sie gelöscht werden können.
Generell sollte im neutrino jetzt eigentlich immer delete[] richtig sein, aber man sollte es nochmals prüfen.
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Richtig blicke ich da auch nicht durch. Ich habe versucht zu verstehen, was da passiert. In rcinput.cpp in den Zeilen 1478 bis 1482 wird mit RC_WithData gearbeitet. Dabei wirdseife hat geschrieben:Das muss nicht richtig sein, weil der rcinput-Code und der infoviewer-Code (der da leider auch mit involviert ist...) unterschiedlich sind.Gaucho316 hat geschrieben:Der folgende Patch schließt drei Speicherlecks in Neutrino im Zusammenhang mit delete. Ich habe ihn aus dem Coolstream GIT übernommen.
Code: Alles auswählen
*data = (neutrino_msg_data_t) p
Code: Alles auswählen
p = new unsigned char[ emsg.dataSize + 1 ];
Edit: In Zeile 2613 von neutrino.cpp habe ich übrigens auch überlegt, ob da nicht delete[] statt delete hingehört. Sicher bin ich mir aber nicht.