Отправлено Igor 06 сентября 2005 г. 12:37

В этом коде компилятор ругается на помеченные звездочкой строчки. Первая ошибка - "error: parse error before "AdcVoltageAverage"". Вторая не ошибка а предупреждение: "warning: integer overflow in expression". Предупреждение не понятно тем, что переполнения быть не может, так как KdVMiddle меньше 65535, вот объявления:
#define OnMiddleValue 180
#define OffValve 240
#define KdVMiddle OffValve*OnMiddleValue

uint16_t AdcVoltageAverage;

Компилирую avr-gcc (GCC) 3.4.3.

Вот сам код:
-----------------------------------------------------------
INTERRUPT (SIG_ADC)
{
uint8_t i;
uint16_t Time,Min,Max;

AdcInterim[ShareSecond]=ADCL+(ADCH<<8);
++ShareSecond;
if (ShareSecond>5)
{
ShareSecond=0;
if (SystemFlags.AdcVoltage==1)
{
AdcVoltageAverage=0;
Min=AdcInterim[0];
Max=Min;
for (i=0;i<6;i++)
{
Time=AdcInterim[i]
* AdcVoltageAverage+=Time;
здесь выдается "error: parse error before "AdcVoltageAverage""
if (Time else if (Time>Max) Max=Time;
}
AdcVoltageAverage=AdcVoltageAverage/6;
Time=Max-Min;
Time=(100*Time)/AdcVoltageAverage;
if (SystemFlags.ValveOn==1)
{
if (AdcVoltageAverage>MaxVoltageValue)
{
ErrorFlags.LargeVoltageOn=1;
}
else
{
ErrorFlags.LargeVoltageOn=0;
if (SystemFlags.ValveFull==1)
{
Time=KdVFull/AdcVoltageAverage;
OnFull=(uint8_t)Time;
OCR0B=OnFull;
}
else
{
* Time=KdVMiddle/AdcVoltageAverage;
здесь выдается "warning: integer overflow in expression"
OnMiddle=(uint8_t)Time;
OCR0B=OnMiddle;
}
}
}
else
{
if (AdcVoltageAverage {
ErrorFlags.SmallVoltageOff=1;
}
else
{
ErrorFlags.SmallVoltageOff=0;
}
}
AdcVoltageNew=AdcAdd;
SystemFlags.AdcVoltage=0;
ADMUX|=(1< }
else
{
AdcCurrentNew=AdcAdd;
SystemFlags.AdcVoltage=1;
ADMUX&= ~(1< }
AdcAdd=0;
SystemFlags.ReadyAdc=1;
}
}

• да уж, у gcc свои ошибки..по идее должен был бы ляпнуть типа =Error : ';' expected= — ы   (06.09.2005 13:44 80.92.98.211, пустое)
  • Дело не в gcc а в моем кривом переводе смысла ошибки. — Igor   (06.09.2005 14:07 217.150.45.49, пустое)
    • 'error: parse error before "AdcVoltageAverage' кто сказал? gcc или вы сами придумали?)) — ы   (06.09.2005 14:11 80.92.98.211, пустое)
      • Не-не-не... Я ничего не придумывал:) parse error before "AdcVoltageAverage" - ошибка допущена до AdcVoltageAverage. Т.е. AdcVoltageAverage здесь не при чем. Ему не повезло, что он находился за ошибкой:) — Igor   (06.09.2005 15:22 217.150.45.49, пустое)
        • да вы меня не поняли, ошибка в том что нет ' ; ' нормальный компилер явно говорит что ее нет gcc же сказал что parse error об этом и речь, если бы он сказал что нет ' ; ' то вам бы не пришлось задаывать этот вопрос)) — ы   (06.09.2005 15:28 80.92.98.211, пустое)
          • ОК. Согласен. — Igor   (06.09.2005 16:51 217.150.45.49, пустое)
• Вау! Мужики спасибо огромное. — Igor   (06.09.2005 13:33 217.150.45.49, пустое)
• перед первой помеченной строкой не хватает ';' — ,,,   (06.09.2005 12:47 85.91.102.34, пустое)
• Ответ: Может после Time=AdcInterim[i] точка с запятой нужны :) — Masterant   (06.09.2005 12:46 212.0.193.134, пустое)
• после Time=AdcInterim[i] где точка с запятой ? А 180*240 это 43200 что больше, чем тип signed int. А компилятор именно за него воспримет это выражение — DASM   (06.09.2005 12:44 217.118.78.1, пустое)
  • PS ну и работать ничего не будет )))) — DASM   (06.09.2005 12:49 217.118.78.1, пустое)
    • Сейчас прогнал этот кусок в отладчике, все нормально. И число правильное грузит - 43200, и ответ правильный. Как бы отключить именно это предупреждение? Смущает. — Igor   (06.09.2005 14:00 217.150.45.49, пустое)
      • попробуй #define OnMiddleValue 180u ну и #define OffValve 240u (то бишь u добавь на конце обеих констант) — DASM   (06.09.2005 14:02 217.118.78.1, пустое)
        • Умница. Все ОК. Спасибо. — Igor   (06.09.2005 14:09 217.150.45.49, пустое)
• Блин в тексте письма удалились лидирующие пробелы. Звездочки это не обращение по адресу, а помеченные строки. Надо было выбрать другой значек для отметки. — Igor   (06.09.2005 12:40 217.150.45.49, пустое)