Re: [kaffe] kaffe/kaffevm/intrp/machine.c warning fix

Sun, 20 Jun 2004 12:39:16 -0700

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.



_______________________________________________
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe

Reply via email to