Error: expected ')' before

[modul (Gast)]

Hallo
Ich hab folgenden Code:

#include <avr/io.h>
#include "pins.h"

int main(void)
{
  set_low(0, DDRD, PORTD);
    while(1)
    {
        //TODO:: Please write your application code 
    }
}

#ifndef PINS_H_
#define PINS_H_

void set_hight(uint8_t pin, uint8_t ddr, uint8_t port);    // Error
void set_low(uint8_t pin, uint8_t ddr, uint8_t port);      // Error

#endif

#include "pins.h"

void set_hight(uint8_t pin, uint8_t ddr, uint8_t port){    // Error
  ddr = ddr | (1<<pin);
  port = port | (1<<pin);
}
void set_low(uint8_t pin, uint8_t ddr, uint8_t port){      // Error
  ddr = ddr && !(1<<pin);
  port = port && !(1<<pin);
}

Nun bekomme ich viermal folgenden Error in den markierten Zeilen:
expected ')' before 'pin'

Was mach ich denn da falsch? Hab schon ein bischen im Internet gesucht 
und bin darauf gestoßen, dass uint8_t unbekannt ist!?

[g457 (Gast)]

> Was mach ich denn da falsch?

Du ignorierst die zugehörigen Warnungen. Die Lösung sollte sein die 
stdint.h mit reinzuholen.

[modul (Gast)]

:) Ok super danke es funktioniert! Aber ich hab leider keine Warnung 
erhalten...
Es sieht jetzt so aus:

#include <avr/io.h>
#include <stdint.h>
#include "pins.h"

int main(void)
{
  set_low(0, DDRD, PORTD);
    while(1)
    {
        //TODO:: Please write your application code 
    }
}

und:

#include <stdint.h>


#ifndef PINS_H_
#define PINS_H_

void set_hight(uint8_t pin, uint8_t ddr, uint8_t port);
void set_low(uint8_t pin, uint8_t ddr, uint8_t port);

#endif

Die andere Datei hab ich nicht veränder...
Wird aber nicht jetzt stdint.h doppelt inkludiert?

[g457 (Gast)]

> Aber ich hab leider keine Warnung erhalten...

Stimmt, hier auch nicht (mehr?). War mir sicher da kommt ein 'warning: 
unknown "uint8_t" .. assuming something something' davor..

> Wird aber nicht jetzt stdint.h doppelt inkludiert?

Jein. Normalerweise(tm) macht man sich keine Sorgen wegen doppelter 
Includes, die sind eh adäquat ∗abgesichert∗ (d.h. 'Ja, wird doppelt 
eingebunden' und 'Nein, es entstehen daraus keine Probleme').

Saubererweise holt sich ∗jeder∗ ∗alle∗ Header rein, die ∗er∗ braucht, 
nicht mehr und nicht weniger. So gesehen braucht obige main die stdint.h 
(noch) nicht.

Unabhängig vom obigen Problem: Überlegs Dir nochmals genau, ob Du 
set_[high,low] wirklich als Funktionen implementieren willst oder nicht 
doch als Präprozessormakros.

HTH und nix für ungut

[modul (Gast)]

Ok danke für die Erklärung!
Habs jetzt über Makros gemacht :)

[Floh (Gast)]

modul schrieb:
> port = port && !(1<<pin);

Wäre sowieso falsch.
Check mal dein Makro, ob derselbe logische Fehler drin ist.
Du benötigst hier bitweises oder und bitweises negieren.

port = port & ~(1<<pin);

[modul (Gast)]

ja hab ich auch bemerkt... trotzdem danke!