[Atmega8] Bruche erklärung zu Fehlermeldung

[Dshing S. (dshing)]

Hi, ich bekomme beim builden die Fehlermeldung:"expected declaration or
statement at end of input" und die Warnung: "'main' is normally a
non-static function"
Ich weiß aber nicht was das nun heißen soll, geschweige dem finde ich
ein Fehler. Ich nutze AVR Studio.
Das merkwürdige ist das es vor ner Woche noch geklappt hat, ich hab es
sogar getestet und alles ging so wie es sollte.
Könnt ihr mir sagen wo das Problem ist?

Hier der Code:

#define F_CPU 1000000UL
#include <avr/io.h>
#include <stdlib.h>
#ifndef F_CPU
#warning "F_CPU war noch nicht definiert, wird nun nachgeholt mit 1000000"
#define F_CPU 1000000UL  
#endif
#define BAUD 4800UL      
#define UBRR_VAL ((F_CPU+BAUD*8)/(BAUD*16)-1)   
#define BAUD_REAL (F_CPU/(16*(UBRR_VAL+1)))    
#define BAUD_ERROR ((BAUD_REAL*1000)/BAUD) 

void HARD (void) {
uint16_t result;
UCSRB |= (1<<TXEN)|(1<<RXEN);
UBRRH = UBRR_VAL >> 8;
UBRRL = UBRR_VAL & 0xFF;
TCCR2 = 0x7A;      
OCR2 =0x01;
ADMUX  |= (0<<REFS1)|(1<<REFS0)|(1<<MUX0)|(0<<MUX1)|(1<<MUX2)|(0<<MUX3);                
ADCSRA |= (1<<ADPS2) |(0<<ADPS1) | (0<<ADPS0)|(1<<ADEN);              
  ADCSRA |= (1<<ADSC);                 
  while (ADCSRA & (1<<ADSC) ) {}      
  result = ADCW;
}
uint8_t ADCWertPC5L () {
ADMUX  |=(1<<MUX0)|(1<<MUX2);
ADMUX &= ~(1<<MUX1);
ADMUX &= ~(1<<MUX3);
ADCSRA |= (1<<ADSC);
while (ADCSRA & (1<<ADSC) ) {}
return ADCL;
}
uint8_t ADCWertPC5H () {
return ADCH;
}
uint8_t ADCWertPC4L () {
ADMUX  |= (1<<MUX2);
ADMUX &= ~(1<<MUX1);
ADMUX &= ~(1<<MUX3);
ADMUX &= ~(1<<MUX0);
ADCSRA |= (1<<ADSC);
while (ADCSRA & (1<<ADSC) ) {}
return ADCL;
}
uint8_t ADCWertPC4H () {
return ADCH;
uint8_t UARTEingang(void)
{
    while (!(UCSRA & (1<<RXC)));   
    return UDR;                   
}
int main(void)
{
  HARD();
  uint8_t ADCWertH;
  uint8_t ADCWertL;
  uint8_t Eingang;
  
    while(1)
    {
  Eingang = UARTEingang ();
  
if(Eingang == 0x01)
  {
    ADCWertL = ADCWertPC5L ();
    ADCWertH = ADCWertPC5H ();
          while (!(UCSRA & (1<<UDRE))) {} 
    UDR = ADCWertL;
     while (!(UCSRA & (1<<UDRE))) {} 
    UDR = ADCWertH;
     while (!(UCSRA & (1<<UDRE))) {} 
    UDR = 0x00;
     }    
if(Eingang == 0x02)
  {
    ADCWertL = ADCWertPC4L ();
    ADCWertH = ADCWertPC4H ();
          while (!(UCSRA & (1<<UDRE))) {} 
    UDR = ADCWertL;
     while (!(UCSRA & (1<<UDRE))) {} 
    UDR = ADCWertH;
     while (!(UCSRA & (1<<UDRE))) {} 
    UDR = 0x00;
     }  }  }

[Peter II (Gast)]

wenn du deinen code sinnvoll formatieren würdest, hättest du vermutlich 
gemerkt das etwas mit den Klammern nicht stimmt.

[Ralph B. (rkhb)]

Dshing Sung schrieb:

> uint8_t ADCWertPC4H () {
> return ADCH;

Hier fehlt noch eine schließende Schweifklammer.

viele grüße
ralph

[Pegel (Gast)]

Und weil ich nett bin, sag ich Dir: "Schau mal bei uint8_t 
ADCWertPC4H(){" nach.

Grüße

[Konrad S. (maybee)]

Finde mal raus, wo deine Funktion ADCWertPC4H endet.

[Helfer (Gast)]

Eine vernünftige Formatierung mit Einrückungen, Leerzeilen und evtl. 
Kommentaren bei schließenden Klammern hilft solche Fehler zu vermeiden.
C-Compiler erkenne solche Fehler zwar, aber wie du gesehen hast, 
kommentieren sie die meist mit wenig hilfreichen bis irreführenden 
Fehlermeldungen.

[Dshing S. (dshing)]

Ahh ok damit hätte ich nun wirklich nicht gerechnete, dass da oben was 
fehlt.
Danke, ich werde in Zukunft versuchen besser zu formatieren. Damit mir 
sowas von allein auffällt.