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 722308 - Plugin should be able to provide deinit function
Plugin should be able to provide deinit function
Status: RESOLVED INCOMPLETE
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal enhancement
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-01-16 02:31 UTC by Andrey Utkin
Modified: 2016-02-21 23:15 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Andrey Utkin 2014-01-16 02:31:42 UTC
This would ehnance plugins ability to cleanup memory and to avoid Valgrind memcheck false positives.
Comment 1 Tim-Philipp Müller 2014-01-16 14:24:14 UTC
We make all plugins resident, so they will never be unloaded.

What would you want to clean up, for example?
Comment 2 Andrey Utkin 2014-01-17 17:37:03 UTC
(In reply to comment #1)
> We make all plugins resident, so they will never be unloaded.
> 
> What would you want to clean up, for example?

Everything currently requiring valgrind suppressions file.
We have gst_deinit() function, so we could free all plugin data at this point.
Comment 3 Sebastian Dröge (slomo) 2014-05-26 12:43:57 UTC
What would this be for example? Can you show a valgrind log?
Comment 4 Tim-Philipp Müller 2016-02-21 23:15:04 UTC
Closing this bug report as no further information has been provided. Please feel free to reopen this bug report if you can provide the information that was asked for in a previous comment.
Thanks!

It's unusual that plugins allocate a lot of things in their plugin_init function, and everything else you can free already via GType and the class system if I'm not mistaken.