Pointer Problem mit SDCC

Nochmal ich :-)

Im Thread http://www.mikrocontroller.net/forum/read-2-40721.html im
Beitrag von Karl Heinz Buchegger (etwa in der Mitte) ist sowas erklärt,
aber funktioniert nicht.

P.S. Das Struct des Menüs hab ich bereits geändert
typedef struct MENU {
  const unsigned char *MenueText;
  unsigned char MenueNo;
  unsigned char NextMenueNo;
  void (*MenueFunction)(unsigned char);
} MENUE_ENTRY;

Nun, du möchtest mit

  Menue[ArrayPos+MenuePos].MenueFunction(MenuePos);

deine Funktion aufrufen.

Wie ist MenuFunction deklariert:

typedef struct MENU {
  const unsigned char *MenueText;
  unsigned char MenueNo;
  unsigned char NextMenueNo;
  void (*MenueFunction)( void );
} MENUE_ENTRY;


Da steht:
Menufunction ist ein Zeiger auf eine Funktion die nichts
zurückliefert (das erste void) und keine Argumente nimmt (das
zweite void). Wenn du also der Funktion Argumente geben willst,
dann musst du das in der Deklaration des Funktionszeigers auch
vereinbaren:

  void (*MenueFunction)( unsigned char );

Jetzt ist MenuFunction ein Zeiger auf eine Funktion die als
erstes Argument einen unsigned char nimmt.

Natürlich müssen jetzt auch alle Funktionen die du an so
einem Funktionszeiger binden willst auch tatsächlich einen
unsigned char annehmen.

Hallo Karl Heinz,
genau das hab ich in meinem 2.Post beschrieben, aber die Fehlermeldung
bleibt dieselbe. Das merkwürdige ist, das die gleiche Zeile bemängelt
wird.

Es funktioniert auch, wenn ich einen Zeiger benutze der nicht in einem
Struct definiert ist.
Ich suche also weiter, wie richtig auf Elemente eines Structs
zugegriffen wird.

Das stimmt an und für sich alles, da das Standard-C ist.

Die Fehlermeldung allerdings deutet darauf hin, dass das
ein compilerspezifisches Problem ist. Der Ausdruck 'reentrant'
ist im C-Standard nicht definiert (auch wenn wir alle wissen
was damit gemeint ist). Da wirst du also im Compiler-Handbuch
suchen muessen, was dem Compiler da nicht passt.

Ich vermite mal, dass es da noch irgendeine 'Dekoration'
für die Funktion gibt, mit dem man dem Compiler mitteilt,
dass die Funktion tatsächlich reentrant ist.

> Es funktioniert auch, wenn ich einen Zeiger benutze der nicht in
> einem Struct definiert ist.

Das ist interessant.

Versuch mal:
  (Menue[ArrayPos+MenuePos].MenueFunction)(MenuePos);

oder

  (*(Menue[ArrayPos+MenuePos].MenueFunction))(MenuePos);

Sollte eigentlich keinen Unterschied machen, wenn ich die
Precedence Regeln noch richtig im Kopf habe, aber es soll
ja auch schon Compilerfehler gegeben haben.

@TheMason

unsigned int foo1( int Arg1 )
{
  return (int)( Arg1 + 0.5 );
}

void main(void)
  {
    unsigned int (*MyFunc)( int );
    MyFunc = foo1;

    i = (*MyFunc)( 4 );   /* Aufruf der Funktion ueber den Pointer */
  }

Das funktioniert, wobei Das Ergebnis natürlich Quatsch ist.

@Karl Heinz
Das Klammern reicht nicht, aber ich habe auch den Verdacht, das es am
Compiler liegt.
Bei Gelegenheit werde ich das mal Visual C testen.

Kannst du dir sparen. In Visual C und allen anderen
C-Compilern auf denen ich bisher gearbeitet habe funktioniert
das klaglos. Muss es auch. Entspricht so den C-Regeln.

Auf einer Website (google: SDCC reentrant)
habe ich folgenden Satz gefunden:

<Zitat>
Avoid calling functions from within an ISR. If you must do this,
declare the function as reentrant (see SDCC manual) which allocates
all local variables in the function on the stack instead of in RAM.
</Zitat>

besonders der letzte Satzteil
"which allocates all local variables in the function on the stack
instead of in RAM" lässt bei mir die Alarmglocken klingen.

Auf jeden Fall: Das ist irgendetwas SDCC spezifisches.

Aus mehreren anderen Websites habe ich mal folgendes abgeleitet:

typedef struct MENU {
  const unsigned char *MenueText;
  unsigned char MenueNo;
  unsigned char NextMenueNo;
  void (*MenueFunction)(unsigned char) reentrant;
} MENUE_ENTRY;

Und dann natürlich noch jede Funktion mit dem Zusatz 'reentrant'
ausstatten.

Super, das funktioniert mit reentrant.

Vielen vielen Dank Karl Heinz