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