On Fri, Jul 31, 2015 at 02:22:03PM -0700, Bill Spitzak wrote: > On Fri, Jul 31, 2015 at 2:02 AM, Elvis Lee <[email protected]> wrote: > > > > > + proxy_destroyed = !!(proxy->flags & > > WL_PROXY_FLAG_DESTROYED); > > + > > + proxy->refcount--; > > + if (proxy_destroyed && !proxy->refcount) > > + free(proxy); > > > > It seems like you don't need the temporary, and could put the test of the > flag directly in the if statement?
I think the way Elvis already did it is better. Jonas _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
