On 01-11-16 13:19, David Sommerseth wrote:
> On 01/11/16 12:28, Gert Doering wrote:
>> 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."Thou shell never waste an opportunity to bikeshed." So, we don't have a shortage of characters here, right? Why not simply 'interval' and 'timeout'? -Steffan
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 [email protected] https://lists.sourceforge.net/lists/listinfo/openvpn-devel
