[kaffe] kaffe/kaffevm/intrp/machine.c warning fix
Guilhem Lavaux
guilhem at kaffe.org
Sun Jun 20 12:29:05 PDT 2004
Arnaud Vandyck wrote:
> in kaffe/kaffevm/exception.h line 99:
> void throwException(struct Hjava_lang_Throwable*) NONRETURNING;
>
> and in kaffe/kaffevm/intrp/machine.c line 270:
> default:
> dprintf("Unknown bytecode %d\n", code[pc]);
> throwException(NEW_LANG_EXCEPTION(VerifyError));
> break;
> #include "kaffe.def"
> }
> }
> end:
> return;
> }
>
> gcc says:
> machine.c:273: warning: will never be executed
>
> does it make sense to remove the break?
>
Hi Arnaud,
I wonder whether we should put 'default' after the #include ... I have
already bad experience with C compilers. I guess we can afford removing
that break.
Cheers,
Guilhem.
More information about the kaffe
mailing list