<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Petr,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 18, 2019 at 5:50 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Li,<br>
<br>
> On Wed, Sep 18, 2019 at 4:24 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br>
<br>
> > ...<br>
<br>
> > > So, maybe that could be as a reason to disable float support for<br>
> > > $LTP_TIMEOUT_MUL?<br>
<br>
> > Maybe I'm missing something, therefore explaining the intent of the code.<br>
> > (int was handled in if tst_is_int "$LTP_TIMEOUT_MUL"; then):<br>
<br>
> I'm not blaming the int/float judgment, there is no problem. My concern is:<br>
<br>
> If the $LTP_TIMEOUT_MUL is float and awk command is missing, how things<br>
> will be going?<br>
<br>
> It will break at:<br>
>     tst_test_cmds awk<br>
> right?<br>
Yes. The concern is described in the commit message (but it should also be in<br>
wiki page I guess): you want float => you need awk. You don't have float, just<br>
set it as int (which will affect also C).<br>
Does it make sense? Is it useful?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It's working, but not elegant. Not sure how many people will go through the documents for such tiny issues.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Is there any possibility to handle float by default, and if no awk supporting, just round up the float to integer and use it do testing automatically?</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Given that break on float number handling, why not declare only support<br>
> integer for $LTP_TIMEOUT_MUL?<br>
Sure, we can do it. But I propose to do it for C as well otherwise setup valid<br>
for C will be breaking shell. But that's a backward incompatibility change.<br>
That's why I suggested IMHO the least intrusive change (but maybe I'm wrong).<br>
Anyone else having strong opinion?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">No more concerns.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> I hope I explained clearly this time, haha ;-)<br>
Sure, sorry to be slow :).<br></blockquote><div> </div><div><span class="gmail_default" style="font-size:small">Never mind, I need to enhance my expression too.</span></div><div><span class="gmail_default" style="font-size:small"></span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Kind regards,<br>
Petr<br>
</blockquote></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>