<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 26, 2016 at 2:33 PM, John Crispin <span dir="ltr"><<a href="mailto:blogic@openwrt.org" target="_blank">blogic@openwrt.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Description of what the patch does is missing<br>
<br>
        John<br>
<div><div class="h5"><br>
On 26/02/2016 03:27, Zhao, Gang wrote:<br>
> Signed-off-by: Zhao, Gang <<a href="mailto:gang.zhao.42@gmail.com">gang.zhao.42@gmail.com</a>><br>
> ---<br>
>  cli.c | 6 +++---<br>
>  1 file changed, 3 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/cli.c b/cli.c<br>
> index f3a041a..c476f35 100644<br>
> --- a/cli.c<br>
> +++ b/cli.c<br>
> @@ -127,7 +127,7 @@ static int ubus_cli_call(struct ubus_context *ctx, int argc, char **argv)<br>
><br>
>  static int ubus_cli_listen(struct ubus_context *ctx, int argc, char **argv)<br>
>  {<br>
> -     static struct ubus_event_handler listener;<br>
> +     struct ubus_event_handler listener;<br></div></div></blockquote><div><br></div><div> Function <span style="color:rgb(80,0,80)">ubus_cli_listen is called only once, and function uloop_run and uloop_doen is called within this function, so I think it's not needed to mark </span></div><div>struct<span style="color:rgb(80,0,80)"> ubus_event_handler listener static.</span></div><div><span style="color:rgb(80,0,80)"><br></span></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div class="h5">
>       const char *event;<br>
>       int ret = 0;<br>
><br>
> @@ -306,7 +306,7 @@ static int usage(const char *prog)<br>
>  }<br>
><br>
><br>
> -struct {<br>
> +static struct {</div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div class="h5">
>       const char *name;<br>
>       int (*cb)(struct ubus_context *ctx, int argc, char **argv);<br>
>  } commands[] = {<br></div></div></blockquote><div><br></div><div>Global struct commands is only used in this file, so it should be marked static.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div class="h5">
> @@ -320,7 +320,7 @@ struct {<br>
>  int main(int argc, char **argv)<br>
>  {<br>
>       const char *progname, *ubus_socket = NULL;<br>
> -     static struct ubus_context *ctx;<br>
> +     struct ubus_context *ctx;<br></div></div></blockquote><div><br></div><div>I think this struct is not needed to be marked static, since it's in the main function.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div class="h5">
>       char *cmd;<br>
>       int ret = 0;<br>
>       int i, ch;<br>
><br>
</div></div>_______________________________________________<br>
openwrt-devel mailing list<br>
<a href="mailto:openwrt-devel@lists.openwrt.org">openwrt-devel@lists.openwrt.org</a><br>
<a href="https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel" rel="noreferrer" target="_blank">https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel</a><br>
</blockquote></div><br></div></div>