GNOME Bugzilla – Bug 789377
global: make possible to set debug-flags dynamically
Last modified: 2018-01-25 17:00:20 UTC
Adding {set,get}_debug_flags functions to the shell global object to make possible to set this easily from looking class, making it easier for developers and users to debug without having to restart the shell with environment variables. Debug flags in main are updated when the "debug-flags" property is changed. I'm keeping this as a string-list so that it's easier to update.
Created attachment 362125 [details] [review] global: make possible to set debug-flags dynamically Adding {set,get}_debug_flags functions to the shell global object to make possible to set this easily from looking class, making it easier for developers and users to debug without having to restart the shell with environment variables. Debug flags in main are updated when the "debug-flags" property is changed. I'm keeping this as a string-list so that it's easier to update.
Created attachment 362126 [details] [review] main: increase the granularity of backtraces in SHELL_DEBUG This is an optional extra, which allows to control the debug flags with better granularity... Not indispensable, but nice to have IMHO.
Review of attachment 362125 [details] [review]: . ::: src/shell-global.h @@ +130,3 @@ const char *property_name); +const char * shell_global_get_debug_flags (ShellGlobal *global); +void shell_global_set_debug_flags (ShellGlobal *global, This file is quite messy in alignment... This is aligned with other void functions, but let me know if you prefer in other way.
Created attachment 362135 [details] [review] global: make possible to set debug-flags dynamically Update the patch based on updates on Bug 789237
Created attachment 362136 [details] [review] main: increase the granularity of backtraces in SHELL_DEBUG Rebasing...
Created attachment 362143 [details] [review] main: increase the granularity of backtraces in SHELL_DEBUG Rebasing on master (and a fix).
Continuing here https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/6