Simple Finite State Machine in C using pointer to function

none of the "state functions" is returning a value when called... so why are you defining the function protocols? you could just define the functions as: "void state1 (void){ code goes here}"

nice