On 01/11/16 12:28, Gert Doering wrote: > Hi, > > On Mon, Oct 31, 2016 at 03:01:28PM +0100, David Sommerseth wrote: >> -.B \-\-keepalive n m >> +.B \-\-keepalive ps pto > > What does "ps" stand for? I find this not much clearer than "n"...
> "pto" sounds like "ping time out", which I find halfway understandable.
>
> Maybe make the first argument "t" for "time"?
>
> Or "i" and "to" for "interval" and "timeout"?
>
> Lots of options for bikeshedding here...
I have no strong opinions here. I thought pi (ping interval), but
considered to close to 3.1415.... ps wasn't goo either, but that's meant
to be "ping seconds" (poor interval alternative, I agree). I thought of
pt and pti as well, but thought that could be somewhat confusing with pto.
pto is as you decoded, Ping Time-Out. I chose 'pto' as just 'to' could
be confusing in the text below ("... timeout is the to argument...").
I also though of 'pin', but that could also be confusing. Even
considered a proper word, but then the man page needs to be rendered in
a way that these keywords are emphasized (like bold/white). Maybe change
ps to int - as in interval]? Or pis (ping interval seconds).
The reasoning for changing n and m to something else was in a hope to it
easier to grasp what each of these arguments really meant - and easier
see them in a bigger context.
But I'm very open for other suggestions; I don't care about the bikeshed
colour as long as it is at least a slight improvement of what we have today.
--
kind regards,
David Sommerseth
signature.asc
Description: OpenPGP digital signature
------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi
_______________________________________________ Openvpn-devel mailing list Openvpn-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openvpn-devel
