On Thu, Nov 15, 2012 at 05:54:26PM +0100, Pavel Březina wrote:
> >Nitpicking, but for this kind of situations I think a switch/case
> >statement would be much more clear (also avoids adding a variable as you
> >use it once):
> >
> >switch(cli_ctx->cli_protocol_version->version) {
> >case 0:
> > DEBUG(..);
> > break;
> >case SSS_SUDO_PROTOCOL_VERSION:
> > break;
> >default:
> > DEBUG("invalid"...);
> > break;
> >}
> >
> >Simo.
> >
>
> OK. New patch is attached.
>
Ack
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
