<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 Thu, Mar 10, 2022 at 4:53 PM <a href="mailto:xuyang2018.jy@fujitsu.com">xuyang2018.jy@fujitsu.com</a> <<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">Hi Li<br>
><br>
><br>
> On Thu, Mar 10, 2022 at 4:38 PM <a href="mailto:xuyang2018.jy@fujitsu.com" target="_blank">xuyang2018.jy@fujitsu.com</a><br>
> <mailto:<a href="mailto:xuyang2018.jy@fujitsu.com" target="_blank">xuyang2018.jy@fujitsu.com</a>> <<a href="mailto:xuyang2018.jy@fujitsu.com" target="_blank">xuyang2018.jy@fujitsu.com</a><br>
> <mailto:<a href="mailto:xuyang2018.jy@fujitsu.com" target="_blank">xuyang2018.jy@fujitsu.com</a>>> wrote:<br>
><br>
>     Hi Li<br>
>      > --- a/lib/newlib_tests/test19.c<br>
>      > +++ b/lib/newlib_tests/test19.c<br>
>      > @@ -23,7 +23,7 @@ static struct tst_test test = {<br>
>      > .needs_root = 1,<br>
>      > .test_all = run,<br>
>      > .setup = setup,<br>
>      > - .save_restore = (const struct tst_path_val const[]) {<br>
>      > + .save_restore = (const struct tst_path_val[]) {<br>
>      > {"?/proc/nonexistent", NULL},<br>
>      > {"!/proc/sys/kernel/numa_balancing", NULL},<br>
>      > {"/proc/sys/kernel/core_pattern", NULL},<br>
><br>
>     Since tst_path_val is a struct array, we should use {NULL, NULL} or {}<br>
>     terminated instead of NULL terminated , otherwise it will report<br>
>     missing<br>
>     braces warning.<br>
><br>
> Ah, right. Sorry!!!<br>
Never mind.<br>
><br>
> Seems I was in so hurry to fix the const issue but overlook this.<br>
I also think you can add a separate patch to fix this comment in doc or <br>
in  tst_test.h for other struct pointers.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">You are right. I will go through the whole part to see if other places need updates.</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">Thanks for finding this :).</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>
          * NULL-terminated array to be allocated buffers.<br>
          */<br>
         struct tst_buffers *bufs;<br>
<br>
         /*<br>
          * NULL-terminated array of capability settings<br>
          */<br>
         struct tst_cap *caps;<br>
<br>
Best Regards<br>
Yang Xu<br>
><br>
> --<br>
> Regards,<br>
> Li Wang<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>