gimp r27999 - in trunk: . app/core

[mitch svn gnome org]

Author: mitch
Date: Sat Feb  7 19:09:55 2009
New Revision: 27999
URL: http://svn.gnome.org/viewvc/gimp?rev=27999&view=rev

Log:
2009-02-07  Michael Natterer  <mitch gimp org>

	* app/core/gimpdrawable.c
	(gimp_drawable_attach_floating_sel)
	(gimp_drawable_detach_floating_sel): call
	gimp_drawable_invalidate_boundary(floating_sel). Fixes missing
	selection update after turning a floating selection into a
	layer. It's called redundantly now when adding or deleting a
	floating selection, but that doesn't hurt much.

	* app/core/gimpfloatingselundo.c (gimp_floating_sel_undo_pop):
	don't call it here because we already call above functions.



Modified:
   trunk/ChangeLog
   trunk/app/core/gimpdrawable.c
   trunk/app/core/gimpfloatingselundo.c

Modified: trunk/app/core/gimpdrawable.c
==============================================================================
--- trunk/app/core/gimpdrawable.c	(original)
+++ trunk/app/core/gimpdrawable.c	Sat Feb  7 19:09:55 2009
@@ -1814,6 +1814,9 @@
 
   g_printerr ("%s\n", G_STRFUNC);
 
+  /*  clear the selection  */
+  gimp_drawable_invalidate_boundary (GIMP_DRAWABLE (floating_sel));
+
   gimp_drawable_sync_source_node (drawable, FALSE);
 
 #ifdef __GNUC__
@@ -1844,4 +1847,7 @@
    *  invalid.
    */
   gimp_viewable_invalidate_preview (GIMP_VIEWABLE (floating_sel));
+
+  /*  clear the selection  */
+  gimp_drawable_invalidate_boundary (GIMP_DRAWABLE (floating_sel));
 }

Modified: trunk/app/core/gimpfloatingselundo.c
==============================================================================
--- trunk/app/core/gimpfloatingselundo.c	(original)
+++ trunk/app/core/gimpfloatingselundo.c	Sat Feb  7 19:09:55 2009
@@ -112,9 +112,6 @@
           gimp_image_set_active_layer (undo->image, floating_layer);
           gimp_image_set_floating_selection (undo->image, floating_layer);
 
-          /*  clear the selection  */
-          gimp_drawable_invalidate_boundary (GIMP_DRAWABLE (floating_layer));
-
           gimp_drawable_attach_floating_sel (gimp_layer_get_floating_sel_drawable (floating_layer),
                                              floating_layer);
         }
@@ -123,9 +120,6 @@
           gimp_drawable_detach_floating_sel (gimp_layer_get_floating_sel_drawable (floating_layer),
                                              floating_layer);
 
-          /*  clear the selection  */
-          gimp_drawable_invalidate_boundary (GIMP_DRAWABLE (floating_layer));
-
           /*  update the pointers  */
           gimp_layer_set_floating_sel_drawable (floating_layer, NULL);
           gimp_image_set_floating_selection (undo->image, NULL);