Re: [cairo-commit] src/cairo-xlib-render-compositor.c

Chris Wilson <[email protected]>
Newsgroups gmane.comp.lib.cairo
Message-ID <[email protected]>
On Mon, Jul 06, 2015 at 02:14:27PM -0700, Bryce Harrington wrote:
>  src/cairo-xlib-render-compositor.c |    8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)

This patch is ridiculous. Just

-cairo_device_destroy(&display->base);
+cairo_device_destroy(priv->device);

with answers on a postcard why the error path is impossible in the first
place.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
-- 
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.