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