c't Projekte - c't-Bot und c't-Sim -
Mailinglisten
[Voriger (Datum)]
[Nächster (Datum)]
[Voriger (Thread)]
[Nächster (Thread)]
[Nach Datum][Nach Thread]
Absender: Achim Pankalla
Datum: Mo, 02.06.2008 17:04:48
In-reply-to:
<F7F3EE9D-548E-4A57-B7BE-A75E393D6576@xxxxxxxxxxxxxxx>
References:
<4842DB09.5000809@xxxxxx> <F7F3EE9D-548E-4A57-B7BE-A75E393D6576@xxxxxxxxxxxxxxx>
hallo,
Timo Sandmann schrieb:
>
> erstmal danke für den Patch, testen muss ich das Ganze dann demnächst
> mal unter Linux.
> Aber was ich bei der neuen Argument-Auswertung nicht verstehe, für den
> Fall 'i' passiert dann nur noch folgendes:
>
> case 'i': {
> /* EEPROM-Init */
> printf("EEPROM soll mit den Daten einer eep-Datei initialisiert
> werden.\n");
> opt_eeprom = 1;
> exit(0);
> }
>
> Das macht aber ja irgendwie keinen Sinn bzw. es passiert dann halt gar
> nix.
der aufruf von init_eeprom_man() erfolgt ja erst nach dem auswerten der
optionen. die variable opt_eeprom wird nun als parameter übergeben.
vorher wurde die funktion fest mit 0 (normaler ablauf, ohne init. des
EEPROM) oder 1 (option -i gesetzt, EEPROM wird mit ct-bot.eep
initialisiert) aufgerufen.
> Wo war denn jetzt das Problem mit der alten Version?
>
> Zur Änderung bei den Inits: Wäre es nicht einfacher, zuerst das EEPROM
> zu initialisieren und dann die Argumente zu prüfen? Dann braucht man
> die Bot-Adresse nicht per globaler Variable zwischenspeichern und sie
> später in main() noch zu setzen. Oder schadet eine doppelte
> Initialisierung des EEPROMs (für den Fall, dass 'i' übergeben wurde)?
ich wollte den eingriff in die ganze initialisierung des ct-bot
möglichst gering halten, um seiteneffekte möglichst auszuschliessen und
den eingriff möglichst klein zu halten.
ich gebe dir natürlich recht, die emulation des eeprom sollte möglichst
früh erfolgen.
eine doppelte initialisierung scheint nicht zu schaden, ich finde
jedoch, man sollte sie vermeiden...
in zuge dieses tests ist mir jedoch aufgefallen, das der patch doch eine
kleine nebenwirkung hat :-[ .
leider kann man die DEBUG meldungen des eeprom-emulators nicht mehr im
Sim-Logfenster sehen, da die initialisierung wohl in bot_2_sim_init()
gemacht wird. :-(
da wird ja wohl eine kleine nachbesserung notwendig. vorschläge??
mit freundlichen grüssen
achim
>
> Grüße,
> Timo
>
>
> _______________________________________________
> ct-bot-entwickler Mailingliste
> ct-bot-entwickler@xxxxxxxxxxxxxxxxx
> http://www.heise.de/bin/newsletter/listinfo/ct-bot-entwickler
>