After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 767155 - base: use MAY_BE_LEAKED flag
base: use MAY_BE_LEAKED flag
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal enhancement
: 1.9.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-06-02 11:08 UTC by Guillaume Desmottes
Modified: 2016-06-02 12:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
use MAY_BE_LEAKED flag (1.55 KB, patch)
2016-06-02 11:09 UTC, Guillaume Desmottes
committed Details | Review

Description Guillaume Desmottes 2016-06-02 11:08:59 UTC
These new flags have been merged to core but not yet released. What is our policy about this? Should we first wait for the next unstable release and bump the dep before starting to use it in master?
Comment 1 Guillaume Desmottes 2016-06-02 11:09:20 UTC
Created attachment 328939 [details] [review]
use MAY_BE_LEAKED flag

With those all tests pass with the leaks tracer.
Comment 2 Tim-Philipp Müller 2016-06-02 11:26:45 UTC
gst-plugins-* master depends on core master, we can merge immediately.
Comment 3 Tim-Philipp Müller 2016-06-02 12:14:56 UTC
commit afdb7d3f32127c3b6c48d04293bdf3114aaaca43
Author: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Date:   Thu Jun 2 13:07:01 2016 +0200

    opusenc, subtitleoverlay: use MAY_BE_LEAKED flag
    
    Flag caps that are cached locally and will never be freed.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=767155