Minor bug in Bootloader?

Hi *,

in atmeaga/ATmegaBOOT_168.c:76-80 you write:

/* the current avr-libc eeprom functions do not support the ATmega168 */
/* own eeprom write/read functions are used instead */
#if !defined(__AVR_ATmega168__) || !defined(__AVR_ATmega328P__)
#include <avr/eeprom.h>
#endif

Isn't this pretty much like "#if 1", given that both defines aren't set at the same time?

Or, in case they are set at the same time, Line 197ff. don't make much sense to me ...

Regards
Yen

Don't know if I'd call it a bug since it doesn't seem to break anything, but it certainly isn't good coding style!

Well, "bug" in the sense of "not doing what it claims to do". (It's most likely not a feature, so ... :wink: )