[Bug 797131] libs: display: wayland: remove NULL check before g_free()

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Sep 12 14:08:25 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=797131

Víctor Manuel Jáquez Leal <vjaquez at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #373620|none                        |reviewed
             status|                            |

--- Comment #5 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
Review of attachment 373620:
 --> (https://bugzilla.gnome.org/review?bug=797131&attachment=373620)

cool! More than I expected!

I guess that would be more than Ok, but if you are willing to continue working
on this, you can also call g_clear_pointer with other deallocators.

For example, you can do the same with wayland, such as here

https://github.com/Igalia/WPEBackend-fdo/blob/da746af2dff6b972c5c4903a132edcafdb768926/src/renderer-backend-egl.cpp#L139

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list