[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