<div dir="ltr">I'll take that out. It's probably a rebasing artifact anyway.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 27, 2015 at 7:43 AM, Frank Henigman <span dir="ltr"><<a href="mailto:fjhenigman@google.com" target="_blank">fjhenigman@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Dec 16, 2015 at 8:37 PM,  <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>> wrote:<br>
> From: Dylan Baker <<a href="mailto:baker.dylan.c@gmail.com">baker.dylan.c@gmail.com</a>><br>
><br>
> This is groundwork for adding a json interface to wflinfo.<br>
> ---<br>
>  src/utils/wflinfo.c | 19 +++++++++++--------<br>
>  1 file changed, 11 insertions(+), 8 deletions(-)<br>
><br>
> diff --git a/src/utils/wflinfo.c b/src/utils/wflinfo.c<br>
> index 268d4b8..905fd90 100644<br>
> --- a/src/utils/wflinfo.c<br>
> +++ b/src/utils/wflinfo.c<br>
> @@ -487,17 +487,18 @@ print_extensions(bool use_stringi)<br>
>      printf("\n");<br>
>  }<br>
><br>
> +static struct {<br>
> +    GLint flag;<br>
> +    char *str;<br>
> +} flags[] = {<br>
> +    { GL_CONTEXT_FLAG_FORWARD_COMPATIBLE_BIT, "FORWARD_COMPATIBLE" },<br>
> +    { GL_CONTEXT_FLAG_DEBUG_BIT, "DEBUG" },<br>
> +    { GL_CONTEXT_FLAG_ROBUST_ACCESS_BIT_ARB, "ROBUST_ACCESS" },<br>
> +};<br>
> +<br>
>  static void<br>
>  print_context_flags(void)<br>
>  {<br>
> -    static struct {<br>
> -        GLint flag;<br>
> -        char *str;<br>
> -    } flags[] = {<br>
> -        { GL_CONTEXT_FLAG_FORWARD_COMPATIBLE_BIT, "FORWARD_COMPATIBLE" },<br>
> -        { GL_CONTEXT_FLAG_DEBUG_BIT, "DEBUG" },<br>
> -        { GL_CONTEXT_FLAG_ROBUST_ACCESS_BIT_ARB, "ROBUST_ACCESS" },<br>
> -    };<br>
>      int flag_count = sizeof(flags) / sizeof(flags[0]);<br>
>      GLint context_flags = 0;<br>
><br>
> @@ -526,6 +527,8 @@ print_context_flags(void)<br>
>          }<br>
>      }<br>
>      printf("\n");<br>
> +<br>
> +    return;<br>
<br>
</div></div>Not a big deal but I don't see the point of adding this return.<br>
<div class="HOEnZb"><div class="h5"><br>
>  }<br>
><br>
>  /// @brief Print out information about the context that was created.<br>
> --<br>
> 2.6.4<br>
><br>
> _______________________________________________<br>
> waffle mailing list<br>
> <a href="mailto:waffle@lists.freedesktop.org">waffle@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/waffle" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/waffle</a><br>
</div></div></blockquote></div><br></div>