<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 Mon, Aug 23, 2021 at 11:05 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@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">We have to erase the last comma because otherwise we pass down one more<br>
empty parameter to the TST_EXP_*_() macros which then causes the FMT<br>
string to be empty and we end up with an empty default message.<br></blockquote><div><br></div><div><br></div><span class="gmail_default" style="font-size:small">P</span>atchset looks good from the code layer though it is a bit complicate<br>in 2/3 <span class="gmail_default" style="font-size:small"></span>stringify handling, but that should be <span class="gmail_default" style="font-size:small">acceptable</span>.</div><div class="gmail_quote"><br><div><div class="gmail_default" style="font-size:small">My only hesitating is about ##__VA_ARGS__, because it says that is still</div><div class="gmail_default" style="font-size:small">as GNU's extension<span class="gmail_default"> but have not got into standard C.</span></div><div class="gmail_default" style="font-size:small"><span class="gmail_default"><br></span></div><div class="gmail_default" style="font-size:small"><span class="gmail_default">So I especially run with GitHub CI and got pretty compiling results, then I have</span></div><div class="gmail_default" style="font-size:small"><span class="gmail_default">a positive </span>attitude on applying these patches unless there is someone who</div><div class="gmail_default" style="font-size:small">blames it broken something in a general (but I guess the possibility is very little).</div><div class="gmail_default" style="font-size:small"><span class="gmail_default"><a href="https://github.com/wangli5665/ltp/runs/3408080506" target="_blank">https://github.com/wangli5665/ltp/runs/3408080506</a><br></span></div></div><div><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_default" style="font-size:small">Feel free to add my reviews:</div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>><br></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>