Re: patch for cleanup protocol

> I'm not qualified to tell if the changes are correct, but I just
> noticed this one thing:
>
> > ";%s=%s;%s=%s;%s=%s;%s=%s;%s=%s;%s=%s;%s=%s;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%s;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d;%s=%d",
> > INI_READONLY,
> > ci->onlyread,
> > - INI_PROTOCOL,
> > - ci->protocol,

...

> If you remove these arguments, you have to remove the corresponding
> (second from start) "%s=%s" pair from the format string, otherwise
> it'll use the wrong variables for the %d's further on.

Yes. Very good point. Thank you Tomas.

> Looks fine to me (haven't tested it though) - however, please add a

I test it. It works good for me. But there can be sill bug (I hope there
is no bug).

> prominent note in dlg_specific.c to remind us not to reuse old parameter
> names/aliases in the future unless it's for the same purpose they were
> originally used for, and list those names of course (well, name in this
> first case).

I try add some note but I don't know if it fits you expectation. If it's
not good for you please change it and send me patch which solve it.

Second try ... (I break in this two advices)
I forgot. I made this patch againist CVS head.

Please review and comment

Luf