Poninter Problem in GCC

[Henning (Gast)]

Mein Compiler gibt mir
../menu.c:13: warning: assignment from incompatible pointer type
aus.

Aber dieser:

unsigned char (*start)(void)

und dieser:

unsigned char (*function_ptr)(unsigned char event)

Poniter sind doch nicht incompatibel.

Was will er von mir ich Blick nicht ganz durch.

[Karl H. (kbuchegg)]

Henning schrieb:
> Mein Compiler gibt mir
> ../menu.c:13: warning: assignment from incompatible pointer type
> aus.
>
> Aber dieser:
>
> unsigned char (*start)(void)
>
> und dieser:
>
> unsigned char (*function_ptr)(unsigned char event)
>
> Poniter sind doch nicht incompatibel.

Doch das sind sie.
Die Funktionen auf die sie zeigen haben unterschiedliche Argumentlisten.
Der eine zeigt auf eine Funktion die keine Argumente nimmt, der andere 
auf eine Funktion die einen unsigned char haben will.

[Henning (Gast)]

Schreibfehler, upps!

[Henning (Gast)]

Ja aber es geht trotzdem. Kann das irgend welche Probleme geben oder 
kann ich da mal drüber hinweg sehen.

[Sven P. (Gast)]

Hängt in weiten Teilen von der Umsetzung des Compilers ab. Warum behebst 
du das Problem nicht einfach?

[Henning (Gast)]

Und wenn ich anstelle von void z.b.
uint8 event;
einsetze gibts keine warining mehr sondern ein error.

Wie bekomm ich das also weg, wie löse ich das?

[Sven P. (Gast)]

Welchen 'error' gibts denn?

Das Problem löst du, indem du die Typen anpasst... (frag dich mal 
selber, was jemand dir jetzt antworten sollte..)

[Henning (Gast)]

Das Problem sollte doch gelöst sein wenn beide gleich sind also:
unsigned char (*start)(unsigned char event)
unsigned char (*function_ptr)(unsigned char event);
ODER?

[Henning (Gast)]

IST ES ABER NICHT

[Maxxie (Gast)]

Also erstmal fehlt da ein ;

Dann: Wenn du jetzt den Typen einfach in der Dekltartion änderst, taucht 
der Fehler evtl. an einer anderen Stelle auf. Sprich, wo kamen die 
unterschiedlichen Typen ursprünglich her. Hast du an diesem Punkt nun 
Typenprobleme?

[Henning (Gast)]

Du meinst da wo ich die benutzte ob ich da jetzt Probleme damit habe:
JA.

Und zwar hier mit
    start_function(show_temperature);

main.c:48: warning: passing argument 1 of 'start_function' from 
incompatible pointer type

[Sven P. (Gast)]

Mann!

Zeig doch endlich den verdammten Quelltext her. Ist doch so schwer nich 
zu verstehn.

[Henning (Gast)]

void start_function(unsigned char (*start)(unsigned char event))
{

 function_ptr = start;

}



void execute_event(unsigned char event)
{

 (*function_ptr)(event);

}

[Johann L. (gjlayde)]

Henning schrieb:
> void start_function(unsigned char (*start)(unsigned char event))
> {
>  function_ptr = start;
> }
>
> void execute_event(unsigned char event)
> {
>  (*function_ptr)(event);
> }

Das gibt des Fehler

error: `function_ptr' undeclared (first use in this function)

Du musst function_ptr definieren, so geht das nicht. Der Compiler 
braucht zumindest eine Deklaration dafür. Eine korrekte Deklaration 
sieht zB so aus:

extern unsigned char (*function_ptr) (unsigned char);

void start_function (unsigned char (*start)(unsigned char))
{
    function_ptr = start;
}

void execute_event (unsigned char event)
{
    function_ptr (event);
}

Johann

[Karl H. (kbuchegg)]

Henning schrieb:
> void start_function(unsigned char (*start)(unsigned char event))
> {
>
>  function_ptr = start;
>
> }
>
>
>
> void execute_event(unsigned char event)
> {
>
>  (*function_ptr)(event);
>
> }

und wo ist da jetzt das incompatible Assignment?
Zeig doch bitte kompletten Code! So schwer ist das nicht.
Da gehören auch Datentypen dazu!

Ein Tip:
Bei der Arbeit mit Funktionspointern ist ein typedef meistens eine 
extrem gute Idee

typedef unsigned char (*CallbackFunc)( unsigned char );

CallbackFunc function_ptr;

void start_function( CallbackFunc start )
{
  function_ptr = start;
}

void execute_event(unsigned char event)
{
  (*function_ptr)(event);
}

unsigned char foo( unsigned char event )
{
  return 0;
}

int main()
{
  start_function( foo );
  execute_event( 1 );
}

[Henning (Gast)]

unsigned char (*function_ptr)(unsigned char event);
ist in der header datei deklariert.

Ich rufe in main das so auf

int main(void)
{

    start_function(show_all);

    for(;;)
    {

     execute_event(event);

    }
}

Und das ist show_all:

void show_all(unsigned char input)
{
 uart_puts("Hallo");
}


Mein Fehler lautet

main.c:48: warning: passing argument 1 of 'start_function' from
incompatible pointer type

[Karl H. (kbuchegg)]

Henning schrieb:
> Du meinst da wo ich die benutzte ob ich da jetzt Probleme damit habe:
> JA.
>
> Und zwar hier mit
>     start_function(show_temperature);
>
> main.c:48: warning: passing argument 1 of 'start_function' from
> incompatible pointer type

Da wird dann show_temperature nicht die signatur haben, die des 
Funktionspointer-Argument zu start_function haben will.

Sprich show_temperature wird nicht so aussehen

unsigned char show_temperature( unsigned char event )
{
}

Die Funktion muss schon von der Gestalt sein, die der Funktionspointer 
vorschreibt. Wie soll denn sonst der Compiler die Argumente über den 
Funktionspointer durchschleifen bzw. den Returnwert abholen, wenn das 
nicht zusammenstimmt?

[Karl H. (kbuchegg)]

Henning schrieb:
> unsigned char (*function_ptr)(unsigned char event);

function_ptr ist ein Pointer auf eine Funktion.
Diese Funktion übernimmt einen unsigned char und sie liefert einen 
unsigned char als Returnwert.


> Und das ist show_all:
>
> void show_all(unsigned char input)

Diese Funktion übernimmt zwar einen unsigned char, aber sie liefert 
keinen Return Wert. Ergo passt diese Funktion nicht zu der Signatur, die 
der Funktionspointer vorschreibt. Denn der Funktionspointer sagt 
explizit, dass die Funktion einen unsigned char liefern muss.

[Henning (Gast)]

Ahja klar jetzt fällt mir das auf stimmt.

Vielen Dank für eure Hilfe