<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Xu,</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">Thanks for reviewing the patches.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com">xuyang2018.jy@cn.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"><span class="gmail_default" style="font-size:small">...</span><br>
> I modify your patch and merged into my ltp own fork.<br>
> see <br>
> <a href="https://github.com/xuyang0410/ltp/commits/Laniel_francis_pipe2_03_v3.fixes" rel="noreferrer" target="_blank">https://github.com/xuyang0410/ltp/commits/Laniel_francis_pipe2_03_v3.fixes</a><br>
> <br>
> ps: I will convert pipe2_01/02.c into new libray and use SAFE_PIPE2 macro.<br>
Can you merge the safe_pipe2 patch(don't need to add this macro into old <br>
safe_macro header) into master? so then I can write new test cases <br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I'm ok with only support it in the new library.</div><br></div><div><div class="gmail_default" style="font-size:small">But if we export the safe macro only to use by new API, we'd better add it into lib/tst_safe_macros.c</div><div class="gmail_default" style="font-size:small">but not lib/safe_macros.c, this patch leaves the 'void (*cleanup_fn) (void)' hook in function safe_pipe2() </div><div class="gmail_default" style="font-size:small">and put in safe_macros.c that is compatible with both API using. </div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Do you think so? if yes I can help to move it to tst_safe_macros.c and apply it.</div></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>