On Thu, Nov 29, 2012 at 01:36:42PM +0100, Jakub Hrozek wrote:
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
Pushed to master.