## c't-Projekte - Mailinglisten

[Voriger (Datum)] [Nächster (Datum)] [Voriger (Thread)] [Nächster (Thread)]
[Nach Datum][Nach Thread]

### Re: [ct-bot] Ticket 167

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
    >

- Antworten:
  - Re: [ct-bot] Ticket 167
    - From: Timo Sandmann

- Referenzen:
  - [ct-bot] Ticket 167
    - From: Achim Pankalla
  - Re: [ct-bot] Ticket 167
    - From: Timo Sandmann

- Voriger (Datum): Re: [ct-bot] Ticket 167, Timo Sandmann
- Nächster (Datum): Re: [ct-bot] Ticket 167, Timo Sandmann
- Voriger (Thread): Re: [ct-bot] Ticket 167, Timo Sandmann
- Nächster (Thread): Re: [ct-bot] Ticket 167, Timo Sandmann
- Indizes:
  - Datum
  - Thread