WinAVR: странный warning, паранойя компилятора или тупость программиста? Опции

[ARV]

Код

    static uint8_t chanel = 0;
    adc[chanel] = ADC;
    chanel = (++chanel) & 7; // на эту строку ругается
    ADMUX = MUX | chanel;
    ADCSRA |= _BV(ADSC);

warning: operation on 'chanel' may be undefined
лично я не вижу ничего подозрительного в этом коде - обычная сумма по модулю 8, т.е. циклически перебираю каналы АЦП...

[ARV]

гм... я свое выражение понимаю так:
1. вычисляется выражение правее знака равенства:
1.1. увеличивается на 1 chanel
1.2. накладывается маска 7 на увеличенное значение.
2. результат заносится в chanel

мне кажется, никакой неопределенности тут нет, другой порядок вычисления просто не может быть. да, chanel обновляется дважды, но где тут криминал? фактически это выражение эквивалентно такой цепочке:

Код

chanel++;
chanel &= 7;

не так ли? тут ведь тоже дважды происходит обновление chanel?

P.S. 2 Модератор: зря перенесли тему... в том разделе aesok наверняка дал бы ответ в соответствии с логикой работы компилятора и стандартами, а сюда он, возможно, и не заглянет...

[demiurg_spb]

Я в своё время пытался так делать, но тоже отказался от этого - не все компиляторы едят это молча

Код

++chanel &= 7;

[Непомнящий Евген...]

Я в своё время пытался так делать, но тоже отказался от этого - не все компиляторы едят это молча

Код

++chanel &= 7;

а зачем? ИМХО, самый прямой путь - chanel = (chanel+1) & 7. Единственное преимущество вашего кода - экономия символов. Зато его хрен поймешь...

[demiurg_spb]

а зачем? ИМХО, самый прямой путь - chanel = (chanel+1) & 7. Единственное преимущество вашего кода - экономия символов. Зато его хрен поймешь...

Да я полностью с Вами согласен. Просто привёл пример до чего додумался раньше, и что сам так не делаю нынче...