<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 Fri, Feb 5, 2021 at 3:44 PM Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com" target="_blank">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">Hi Li<br>
> Hi Xu,<br>
><br>
> Thanks for your work.<br>
><br>
>     --- /dev/null<br>
>     +++ b/include/tst_mallinfo.h<br>
><br>
><br>
> Do we really need to export this function into the LTP library?<br>
> (I assumed no other tests will using this lib function anymore)<br>
Maybe we can add tst_print_mallinfo2 in the future and mallinfo2() case<br>
can use this header.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Or put it into the tst_memutils.h, I guess this header file is a general-purpose for providing kinds of memory helpers.</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">
<br>
><br>
> So I prefer to create a common header file as:<br>
> ../syscalls/mallinfo/mallinfo_common.h,<br>
> and just use it locally.<br>
><br>
> If you're hoping mallopt01.c also benefits from it, only need to<br>
> build something like mallinfo_commo.o via Makefile and include<br>
> it in mallopt01.c by "../mallinfo/mallinfo_common.h".<br>
Just want to avoid random makefile. I am not sure which way is better. <br>
Let's listen advise from other maintainers.<br></blockquote><div> </div><div class="gmail_default" style="font-size:small">Sure, my pleasure to hear more voice~</div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>