Hi Volker, On Sat, Nov 10, 2007 at 09:37:24PM +0100, Volker Lendecke wrote: > +const char *ndr_errstr(enum ndr_err_code err) > +{ > + switch (err) { > + case NDR_ERR_SUCCESS: > + return "NDR_ERR_SUCCESS"; > + break; ^^ the break here after the return shouldn't be necessary. Other than that, looks good. Cheers, Jelmer