array of function pointers --- syntax error!?

[damian birchler]

What's wrong about this:

22: static void (*)(void) instruction_tab le[] = {
jnz,
halt,
mv,
add,
mul,
mv_reg,
add_reg,
mul_reg,
pop,
push,
};

The compiler doesn't complain about wrong types of the functions, it says:

interpreter.c:2 2: syntax error before `)'
interpreter.c: In function `execute_instru ction':
interpreter.c:6 6: `instruction_ta ble' undeclared (first use in this function)
interpreter.c:6 6: (Each undeclared identifier is reported only once
interpreter.c:6 6: for each function it appears in.)

Thanks a lot
damian

[Joona I Palaste]

damian birchler <da************ *@bluewin.ch> scribbled the following:

What's wrong about this: 22: static void (*)(void) instruction_tab le[] = {
jnz,
halt,
mv,
add,
mul,
mv_reg,
add_reg,
mul_reg,
pop,
push,
}; The compiler doesn't complain about wrong types of the functions, it says: interpreter.c:2 2: syntax error before `)'
interpreter.c: In function `execute_instru ction':
interpreter.c:6 6: `instruction_ta ble' undeclared (first use in this function)
interpreter.c:6 6: (Each undeclared identifier is reported only once
interpreter.c:6 6: for each function it appears in.)

The error is most likely in the function execute_instruc tion, not in the
above declaration. Please post the code to execute_instruc tion.

--
/-- Joona Palaste (pa*****@cc.hel sinki.fi) ------------- Finland --------\
\-------------------------------------------------------- rules! --------/
"There's no business like slow business."
- Tailgunner

[Kenneth Brody]

damian birchler wrote:

What's wrong about this:

22: static void (*)(void) instruction_tab le[] = { [...]
The compiler doesn't complain about wrong types of the functions, it says:

[...]

"instruction_ta ble" is the array of pointers. Therefore, you need:

static void (*instruction_t able[])(void) = ...

Or, as I prefer to do:

typedef void (*VoidFunc)(voi d);
static VoidFunc instruction_tab le[] = ...

--
+-------------------------+--------------------+-----------------------------+
| Kenneth J. Brody | www.hvcomputer.com | |
| kenbrody/at\spamcop.net | www.fptech.com | #include <std_disclaimer .h> |
+-------------------------+--------------------+-----------------------------+
Don't e-mail me at: <Th************ *@gmail.com>

[Nils O. Selåsdal]

On Wed, 20 Oct 2004 11:45:43 -0700, damian birchler wrote:

What's wrong about this:

22: static void (*)(void) instruction_tab le[] = {
jnz,
halt,
mv,
add,
mul,
mv_reg,
add_reg,
mul_reg,
pop,
push,
};

The compiler doesn't complain about wrong types of the functions, it says:

interpreter.c:2 2: syntax error before `)'
interpreter.c: In function `execute_instru ction':
interpreter.c:6 6: `instruction_ta ble' undeclared (first use in this function)
interpreter.c:6 6: (Each undeclared identifier is reported only once
interpreter.c:6 6: for each function it appears in.)

Thanks a lot

Did you mean
static void *(*instruction_ table[])(void) = { ... }
?

[Dave Vandervies]

In article <29************ **************@ posting.google. com>,
damian birchler <da************ *@bluewin.ch> wrote:

What's wrong about this:

22: static void (*)(void) instruction_tab le[] = {
<snip>
The compiler doesn't complain about wrong types of the functions, it says:

interpreter.c: 22: syntax error before `)'
You have a syntax error.

Quoth cdecl:
cdecl> declare instruction_tab le as array of pointer to function (void) returning void
void (*instruction_t able[])(void )

So I think you might have wanted to say:
static void (*instruction_t able[])(void) = {

'Tmight also be worth using a typedef to make things a bit simpler:

typedef void (*instruction_f unc_ptr)(void);
static instruction_fun c_ptr instruction_tab le[] = {/*stuff*/};

interpreter. c: In function `execute_instru ction':
interpreter.c: 66: `instruction_ta ble' undeclared (first use in this function)
interpreter.c: 66: (Each undeclared identifier is reported only once
interpreter.c: 66: for each function it appears in.)

The compiler attempted to recover from the syntax error so it could warn
you about other things, but this particular error is bogus, because it
depends on the part that the compiler skipped doing the error recovery
in the declaration.
dave

--
Dave Vandervies dj******@csclub .uwaterloo.ca
But they're not strings. Not being strings doesn't stop them being useful.
After all, ints aren't strings, yet they are very useful indeed!
--Richard Heathfield in comp.lang.c

[jacob navia]

damian birchler wrote:

What's wrong about this:

22: static void (*)(void) instruction_tab le[] = {
jnz,
halt,
mv,
add,
mul,
mv_reg,
add_reg,
mul_reg,
pop,
push,
};

The compiler doesn't complain about wrong types of the functions, it says:

interpreter.c:2 2: syntax error before `)'
interpreter.c: In function `execute_instru ction':
interpreter.c:6 6: `instruction_ta ble' undeclared (first use in this function)
interpreter.c:6 6: (Each undeclared identifier is reported only once
interpreter.c:6 6: for each function it appears in.)

Thanks a lot
damian

Here is something that works

void jnz(void);void halt(void);void mv(void);void add(void);
void mul(void);void mv_reg(void);vo id add_reg(void); void mul_reg(void);
void pop(void);void push(void);

typedef void (*voidfn)(void) ;
static voidfn instruction_tab le[] = {
jnz,
halt,
mv,
add,
mul,
mv_reg,
add_reg,
mul_reg,
pop,
push,
};

C can be horrible, and arrays of function pointers belong to these
things that are unnecesarily difficult in the language.