<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Xu,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 29, 2021 at 5:27 PM Yang Xu <<a href="mailto:xuyang2018.jy@fujitsu.com">xuyang2018.jy@fujitsu.com</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">1) test <span class="gmail_default" style="font-size:small"></span>TST_EXP_FD_SILENT macro in test_macros01.c<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I'm thinking maybe we should add more info to indicate the TST_PASS result</div><div class="gmail_default" style="font-size:small">from which <span class="gmail_default"></span>TST_EXP_FD_SILENT? Because as <span class="gmail_default"></span>TST_EXP_FD_SILENT will</div><div class="gmail_default" style="font-size:small">keep silent, the only returned looks strange to us.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">e.g.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">        TST_EXP_FD_SILENT(pass_fd(), "%s", "SCALL");<br>-       tst_res(TINFO, "TST_PASS = %i", TST_PASS);<br>+       tst_res(TINFO, "TST_PASS = %i from TST_EXP_FD_SILENT(pass_fd, ...)", TST_PASS);<br></div><br></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">
2) test TST_EXP_FAIL2 macro in test_macros02.c<br>
</blockquote><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">3) test TST_EXP_PASS_SILENT macro in test_macros03.c<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">here as well.</div><div class="gmail_default" style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
4) add test_macros04.c to test TST_EXP_PID and TST_EXP_PID_SILENT macros<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">and here.</div><div><br></div><div class="gmail_default" style="font-size:small">Btw, the remaining part looks good to me.</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>