g_return_if_fail() earlier to clarify things (There was a bug in 1.2 which

2002-11-17  Michael Natterer  <mitch@gimp.org>

	* app/core/gimpcontext.c (gimp_context_set_default_colors,
	gimp_context_swap_colors): g_return_if_fail() earlier to clarify
	things (There was a bug in 1.2 which was fixed only accidentially
	in HEAD).

ChangeLog

+2002-11-17  Michael Natterer  <mitch@gimp.org>
+
+	* app/core/gimpcontext.c (gimp_context_set_default_colors,
+	gimp_context_swap_colors): g_return_if_fail() earlier to clarify
+	things (There was a bug in 1.2 which was fixed only accidentially
+	in HEAD).
+
 2002-11-17  Michael Natterer  <mitch@gimp.org>
 	* app/widgets/gimpdnd.h: added GTK_TARGET_SAME_APP flag

app/core/gimpcontext.c

@@ -1856,9 +1856,10 @@ gimp_context_set_default_colors (GimpContext *context)
   GimpRGB      fg;
   GimpRGB      bg;
 
+  g_return_if_fail (GIMP_IS_CONTEXT (context));
+
   bg_context = context;
 
-  g_return_if_fail (GIMP_IS_CONTEXT (context));
   context_find_defined (context, GIMP_CONTEXT_FOREGROUND_MASK);
   context_find_defined (bg_context, GIMP_CONTEXT_BACKGROUND_MASK);
 
@@ -1876,9 +1877,10 @@ gimp_context_swap_colors (GimpContext *context)
   GimpRGB      fg;
   GimpRGB      bg;
 
+  g_return_if_fail (GIMP_IS_CONTEXT (context));
+
   bg_context = context;
 
-  g_return_if_fail (GIMP_IS_CONTEXT (context));
   context_find_defined (context, GIMP_CONTEXT_FOREGROUND_MASK);
   context_find_defined (bg_context, GIMP_CONTEXT_BACKGROUND_MASK);