<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 26, 2020 at 5:18 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">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>
...<br>
> > obvious fix which I'd normally merge without sending to ML.<br>
> > But there are 19 tests which use tst_res(TBROK, and at least some of<br>
> > them are affected by this (i.e. don't unconditionally report any result<br>
> > before tst_res(TBROK), so we might want to fix it different way.<br>
<br>
> +1 for this fix.<br>
Thanks for your review.<br>
<br>
> By the way, do you think it makes sense to add build check parameters for<br>
> tst_res() too? Just like what we do for tst_brk() in commit 0738e3753c51.<br>
<br>
IMHO if we agree that tst_res(TBROK) is harmful and should be avoided entirely,<br></blockquote><div><span class="gmail_default" style="font-size:small">I agree.</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">
I'd be for adding this check.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">In personally, I'd let tst_res() only accept 'TPASS, TFAIL, TINFO, TCONF, TWARN'.</div></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>