This is never reached (while (1)), but doesn't hurt.  Does the compiler/analyser shut up if you remove the return ""; instead?