<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 27, 2017 at 1:53 AM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<span class=""><br>
On 26 September 2017 at 23:55, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
>  static void<br>
> -wsi_wl_display_destroy(struct wsi_wl_display *display)<br>
> +wsi_wl_display_ref(struct wsi_wl_display *display)<br>
> +{<br>
> +   display->refcount++;<br>
> +}<br>
<br>
</span>Better:<br>
static struct wsi_wl_display *<br>
wsi_wl_display_ref(struct wsi_wl_display *display)<br>
{<br>
   display->refcount++;<br>
   return display;<br>
}<br>
<br>
Makes it much more difficult to get it wrong. :)<br></blockquote><div><br></div><div>Yeah.  I thought about doing that the first time around but decided against it for no good reason.  You've pushed me over the edge back into sanity. :) <br></div></div></div></div>