--with-updatehttpprefix=disable -> Änderung

[Striper]

Es geht um diese Änderung: http://cvs.tuxbox-cvs.sourceforge.net/l ... 00179.html

neutrino_menu.cpp:

Könnte bitte jmd. das erste

Code: Alles auswählen

#ifndef DISABLE_INTERNET_UPDATE

oberhalb von:

Code: Alles auswählen

mtdexpert->addItem(GenericMenuSeparatorLine);

setzen? Dann wäre das Konsistent zu den anderen Menüs in Neutrino. So wie jetzt sieht das komisch aus.

[Barf]

vielleicht leicht OT:

@seife: Hast du "benutze positive Logik" ("use positive logic") irgendwann gehört?

#ifdef ENABLE_DINGSBUMS is viel besser als #ifnotdef DISABLE_DINGSBUMS.

(Bitte als konstuktive Kritik sehen )

[flasher]

setzen? Dann wäre das Konsistent zu den anderen Menüs in Neutrino. So wie jetzt sieht das komisch aus.

In welcher Form sieht das denn "komisch" aus?

Wenn ich mit den Code anschaue dann kann ich mir das Aussehen so vorstellen:

Mit UPDATEPREFIX

zurück
--------------
readflash
writeflash
--------------
readmtd
writemtd
--------------
update urls
expertenfunktion
u.s.w

Ohne UPDATEPREFIX

zurück
--------------
readflash
writeflash
--------------
readmtd
writemtd
--------------
expertenfunktion
u.s.w

Komisch sieht das nicht aus.
Oder möchte Du zwischen "einzelne Partitionen schreiben" und "Experten Funktion" einfach den Trennen nicht haben?

Stelle ich mir das falsch vor? Dann mach mal ein Bild.

Gruß

[Striper]

Seit wann ist mtdexpert = updateSettings? Du vermischt da verschiedene Menüs...

So wie es jetzt ist wird noch eine Seperator Line gezeichnet am Menüende und danach kommt nix mehr. Mit meiner Änderung wird die Seperator Line am Ende des Menüs nicht gezeichnet und das ganze sieht wieder "normal" aus.

[flasher]

Seit wann ist mtdexpert = updateSettings? Du vermischt da verschiedene Menüs...

So wie es jetzt ist wird noch eine Seperator Line gezeichnet am Menüende und danach kommt nix mehr. Mit meiner Änderung wird die Seperator Line am Ende des Menüs nicht gezeichnet und das ganze sieht wieder "normal" aus.

Schlag mich doch
Ich hatte mich zu nachtschlafender Zeit einfach nur verlesen.
Bei dieser "bescheidenen" Anordnung ja kaum verwunderlich...

Zeile 700-703 sind Updatesettings 707-722 dann mtdexpert und ab 724 fangen dann gehen wieder die Updatesettings weiter... Was ein Blödsinn...

Jetzt habe ich auch erkannt was Du meinst. 719 an 717 und die 717 rustcht dann automatisch in die 718...

Korrekt?

Gruß

[Striper]

Schlag mich doch :o

Ist offensichtlich ein wenig forsch rübergekommen. Das war nicht meine Absicht.

Thema ist nun eh erledigt, da es im CVS geändert wurde. THXle!

[flasher]

Thema ist nun eh erledigt, da es im CVS geändert wurde. THXle!

Ich weiß, mein CVS Name nennt sich ecosys. Warum? Habe ich vergessen...

Gruß