<div dir="ltr">Sounds good to me. Updated patch attached.<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 14, 2015 at 1:34 AM, Petr Spacek <span dir="ltr"><<a href="mailto:pspacek@redhat.com" target="_blank">pspacek@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 14.9.2015 07:23, Jan Cholasta wrote:<br>
> IMO it does, because saying just "-1 is default" is not entirely correct and<br>
> "0 is default" would be confusing, as you pointed out. You might say "0 or -1<br>
> is unlimited" if you think it's clearer.<br>
<br>
</span>my +1 to "0 or -1 is unlimited" variant<br>
<br>
Petr^2 Spacek<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> On 10.9.2015 18:39, Gabe Alford wrote:<br>
>> Oops.. replied without the list.<br>
>><br>
>> Reason I said -1 is because users might be confused if they enter `ipa<br>
>> config-mod --searchtimelimit=0`, and both `ipa user-show` and the webui<br>
>> show -1 instead of 0. I wonder if -1 makes more sense in that regard?<br>
>> Thoughts? Does "<= 0 is unlimited" make more sense?<br>
>><br>
>> Thanks,<br>
>><br>
>> Gabe<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Manage your subscription for the Freeipa-devel mailing list:<br>
<a href="https://www.redhat.com/mailman/listinfo/freeipa-devel" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/freeipa-devel</a><br>
Contribute to FreeIPA: <a href="http://www.freeipa.org/page/Contribute/Code" rel="noreferrer" target="_blank">http://www.freeipa.org/page/Contribute/Code</a><br>
</font></span></blockquote></div><br></div></div></div>