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 649517 - Change 'debugexit' to quit main loop rather than exit(), add cleanup
Change 'debugexit' to quit main loop rather than exit(), add cleanup
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-05-05 22:16 UTC by Colin Walters
Modified: 2011-05-19 19:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Change 'debugexit' to quit main loop rather than exit(), add cleanup (3.32 KB, patch)
2011-05-05 22:16 UTC, Colin Walters
committed Details | Review

Description Colin Walters 2011-05-05 22:16:32 UTC
A new envrionment variable GNOME_SHELL_ENABLE_CLEANUP is added which
causes us to attempt freeing global data.  The reason this isn't
enabled by default is that it's a waste of time at best, and at
worst in corner cases could cause crashes which would fill up
crash databases.  Better to leave it as a developer-only tool.

Start stubbing out some cleanup in ShellGlobal.
Comment 1 Colin Walters 2011-05-05 22:16:35 UTC
Created attachment 187325 [details] [review]
Change 'debugexit' to quit main loop rather than exit(), add cleanup
Comment 2 Dan Winship 2011-05-19 16:53:54 UTC
Comment on attachment 187325 [details] [review]
Change 'debugexit' to quit main loop rather than exit(), add cleanup

ok
Comment 3 Colin Walters 2011-05-19 19:34:49 UTC
Honestly this is sort of unfinished; the goal is to make valgrind useful obviously, but we have a lot more to do there.  On the other hand it's not harmful.

So if you think it's OK I'll put it in.
Comment 4 Colin Walters 2011-05-19 19:37:16 UTC
Attachment 187325 [details] pushed as a8baf4a - Change 'debugexit' to quit main loop rather than exit(), add cleanup