<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Richard Palethorpe <<a href="mailto:rpalethorpe@suse.de">rpalethorpe@suse.de</a>> wrote:</div></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>> > +  int max_iteration_runtime;<br>
>> <br>
>> It's not immediately clear if iteration refers to the inner test loop<br>
>> (e.g. Fuzzy Sync) or the outer loop performed by adding '-i N'. Perhaps<br>
>> it would be better to call it max_runtime and document that it is scaled<br>
>> by '-i N'?<br>
><br>
> It's not only the -i N paramater, it's carthesian product of:<br>
><br>
>       -i parameter x all_filesystems x test_variants<br>
><br>
> And every single instance of that product is limited by the runtime<br>
> value, which is the reason I want to have iteration explicitly in the<br>
> name. Maybe we should call it instance instead or whatever else that may<br>
> be more fitting.<br>
<br>
I suppose it is the innermost runtime, so I would vote for inner_runtime<br>
or just runtime. However whatever it is called it will require<br></blockquote><div> </div><div><span class="gmail_default" style="font-size:small"></span>I do really have the same feelings here. I'm even afraid 'iteration' will</div><div class="gmail_default">confused people who are not familiar with ltp-library.</div><div class="gmail_default"><br></div><div class="gmail_default">From my comprehension, max_iteration_runtime takes effect on every</div><div class="gmail_default">onefold test iteration, or maybe just directly rename to 'max_test_runtime'</div><div class="gmail_default">(or max_runtime) with explanation in code comments.</div><div><span class="gmail_default" style="font-size:small"><br></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">
explanation. What is considered to be an iteration, instance, inner, test<br>
etc. is arbitrary.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Yes, includes "onefold" which hovers in my mind.</div><br></div></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>