> I'm not convinced by your argument, and I don't think I'm going to > change your mind, so I propose a comprimise. > > Why not enclose the default: block with #ifdef DEBUG/#endif. That > way during driver development, we have the extra check and during > "normal" runtime you don't have the overhead? Solution #3: I've commited a different fix which doesn't imply changes in error paths, so I have no reason to change what you did and everyone's happy ;) -- Jean Delvare http://www.ensicaen.ismra.fr/~delvare/