<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 Sat, Nov 14, 2020 at 4:28 AM 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">> In order to simplify common return value checks.<br>
<br>
Great improvement, thanks!<br>
<br>
The only concern I have is for tests which does not end 0<br>
as it's not obvious from the output (without looking into the test source)<br>
that everything it's working, but not a big deal.<br></blockquote><div> </div><div><div class="gmail_default" style="font-size:small">+1</div><div class="gmail_default" style="font-size:small">I agree to add an expected value check for TEST_PASS(), e.g</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">#define TEST_PASS(SCALL, EXPECTED, ...)<br></div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
FYI I plan to send a patchset which runs tests in lib/newlib_tests/.<br>
First I thought simple check for 0 would be ok, but with non-zero results it<br>
might be good to not only check expected return value but maybe for whole<br>
output (the same approach we have unfinished in patchset from Christian Lanig).<br>
<br>
Kind regards,<br>
Petr<br>
<br>
-- <br>
Mailing list info: <a href="https://lists.linux.it/listinfo/ltp" rel="noreferrer" target="_blank">https://lists.linux.it/listinfo/ltp</a><br>
<br>
</blockquote></div><br clear="all"><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>