summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: ebc3a57)
raw | patch | inline | side by side (parent: ebc3a57)
author | buliabyak <buliabyak@users.sourceforge.net> | |
Sun, 18 Mar 2007 02:01:19 +0000 (02:01 +0000) | ||
committer | buliabyak <buliabyak@users.sourceforge.net> | |
Sun, 18 Mar 2007 02:01:19 +0000 (02:01 +0000) |
src/flood-context.cpp | patch | blob | history |
diff --git a/src/flood-context.cpp b/src/flood-context.cpp
index 367e56a3dd1af334cb827223c2611d1b25f1c029..648d00bcbc5e055d754b2a7f51a0592f242e11a6 100644 (file)
--- a/src/flood-context.cpp
+++ b/src/flood-context.cpp
@@ -779,15 +779,20 @@ static gint sp_flood_context_root_handler(SPEventContext *event_context, GdkEven
if (!(event->button.state & GDK_CONTROL_MASK)) {
// set "busy" cursor
desktop->setWaitingCursor();
+
+ if (SP_IS_EVENT_CONTEXT(event_context)) {
+ // Since setWaitingCursor runs main loop iterations, we may have already left this tool!
+ // So check if the tool is valid before doing anything
+
+ sp_flood_do_flood_fill(event_context, event);
- sp_flood_do_flood_fill(event_context, event);
-
- // restore cursor when done; note that it may already be different if e.g. user
- // switched to another tool during interruptible tracing or drawing, in which case do nothing
- if (desktop->waiting_cursor)
- sp_event_context_update_cursor(event_context);
+ // restore cursor when done; note that it may already be different if e.g. user
+ // switched to another tool during interruptible tracing or drawing, in which case do nothing
+ if (desktop->waiting_cursor)
+ sp_event_context_update_cursor(event_context);
- ret = TRUE;
+ ret = TRUE;
+ }
}
}
break;