Revision a9451570b92d0f29b2e120ce02c4301e9ed3ced6

markdoliner at pidgin.im markdoliner at pidgin.im
Tue Apr 3 02:30:36 EDT 2007


o   -----------------------------------------------------------------
|   Revision: a9451570b92d0f29b2e120ce02c4301e9ed3ced6
|   Ancestor: fa201fe30f55d6bb7c3968fde6784153a44c13be
|   Author: markdoliner at pidgin.im
|   Date: 2007-04-03T06:30:22
|   Branch: im.pidgin.pidgin
|   
|   Modified files:
|           pidgin/gtkdebug.c
|   
|   ChangeLog: 
|   
|   Check if debug_win is != NULL before checking the preference.  This
|   should avoid a hash table lookup when debugging is turned off.  I think
|   debug_win will ALWAYS be non-null when debugging is on and ALWAYS be
|   null when debugging is off, but I'm not positive and I didn't want
|   to break anything, so I left both checks in there.
|   
|   ============================================================
|   --- pidgin/gtkdebug.c	950e991a266244378cc8efc2147236f4e0019529
|   +++ pidgin/gtkdebug.c	15b6f15cf85543d662e57e49062b11dac29b8818
|   @@ -1046,8 +1046,8 @@ pidgin_debug_print(PurpleDebugLevel leve
|    	gchar *ts_s;
|    	gchar *esc_s, *cat_s, *tmp, *s;
|    
|   -	if (!purple_prefs_get_bool("/purple/gtk/debug/enabled") ||
|   -	    (debug_win == NULL))
|   +	if (debug_win == NULL ||
|   +		!purple_prefs_get_bool("/purple/gtk/debug/enabled"))
|    	{
|    		return;
|    	}
|   @@ -1107,8 +1107,8 @@ pidgin_debug_is_enabled(PurpleDebugLevel
|    static gboolean
|    pidgin_debug_is_enabled(PurpleDebugLevel level, const char *category)
|    {
|   -	return (purple_prefs_get_bool("/purple/gtk/debug/enabled") &&
|   -			debug_win != NULL);
|   +	return (debug_win != NULL &&
|   +			purple_prefs_get_bool("/purple/gtk/debug/enabled"));
|    }
|    
|    static PurpleDebugUiOps ops =

To get the patch for this revision, please do this:
mtn log --last 1 --diffs --from a9451570b92d0f29b2e120ce02c4301e9ed3ced6


More information about the Commits mailing list