added local "gboolean press" variables to improve readability.

2006-09-13  Michael Natterer  <mitch@gimp.org>

	* app/tools/gimptool.c (gimp_tool_set_active_modifier_state):
	added local "gboolean press" variables to improve readability.

2
This commit is contained in:
Michael Natterer 2006-09-12 22:55:03 +00:00 committed by Michael Natterer
parent a37883664a
commit 3bc9a7af2d
2 changed files with 21 additions and 16 deletions

View file

@ -1,3 +1,8 @@
2006-09-13 Michael Natterer <mitch@gimp.org>
* app/tools/gimptool.c (gimp_tool_set_active_modifier_state):
added local "gboolean press" variables to improve readability.
2006-09-12 Michael Natterer <mitch@gimp.org>
* plug-ins/common/tiff.c: added a parargaph of the original
@ -64,7 +69,7 @@
* app/paint/gimpperspectiveclone.[ch]: derive from GimpClone and
remove everything that is already imeplemented in the parent
classes. Particullarly, removed motion() implementation in favor
classes. Particularly, removed motion() implementation in favor
of a get_source() implementation and simply use GimpClone's
motion(). Also refuse to work on indexed drawables.

View file

@ -590,21 +590,21 @@ gimp_tool_set_active_modifier_state (GimpTool *tool,
if ((tool->active_modifier_state & GDK_SHIFT_MASK) !=
(state & GDK_SHIFT_MASK))
{
gboolean press = state & GDK_SHIFT_MASK;
#ifdef DEBUG_ACTIVE_STATE
g_printerr ("%s: SHIFT %s\n", G_STRFUNC,
(state & GDK_SHIFT_MASK) ? "pressed" : "released");
press ? "pressed" : "released");
#endif
if (! (state & GDK_SHIFT_MASK) &&
(tool->button_press_state & GDK_SHIFT_MASK))
if (! press && (tool->button_press_state & GDK_SHIFT_MASK))
{
tool->button_press_state &= ~GDK_SHIFT_MASK;
}
else
{
gimp_tool_active_modifier_key (tool, GDK_SHIFT_MASK,
(state & GDK_SHIFT_MASK) ? TRUE : FALSE,
state,
press, state,
display);
}
}
@ -612,21 +612,21 @@ gimp_tool_set_active_modifier_state (GimpTool *tool,
if ((tool->active_modifier_state & GDK_CONTROL_MASK) !=
(state & GDK_CONTROL_MASK))
{
gboolean press = state & GDK_CONTROL_MASK;
#ifdef DEBUG_ACTIVE_STATE
g_printerr ("%s: CONTROL %s\n", G_STRFUNC,
(state & GDK_CONTROL_MASK) ? "pressed" : "released");
press ? "pressed" : "released");
#endif
if (! (state & GDK_CONTROL_MASK) &&
(tool->button_press_state & GDK_CONTROL_MASK))
if (! press && (tool->button_press_state & GDK_CONTROL_MASK))
{
tool->button_press_state &= ~GDK_MOD1_MASK;
}
else
{
gimp_tool_active_modifier_key (tool, GDK_CONTROL_MASK,
(state & GDK_CONTROL_MASK) ? TRUE : FALSE,
state,
press, state,
display);
}
}
@ -634,21 +634,21 @@ gimp_tool_set_active_modifier_state (GimpTool *tool,
if ((tool->active_modifier_state & GDK_MOD1_MASK) !=
(state & GDK_MOD1_MASK))
{
gboolean press = state & GDK_MOD1_MASK;
#ifdef DEBUG_ACTIVE_STATE
g_printerr ("%s: ALT %s\n", G_STRFUNC,
(state & GDK_MOD1_MASK) ? "pressed" : "released");
press ? "pressed" : "released");
#endif
if (! (state & GDK_MOD1_MASK) &&
(tool->button_press_state & GDK_MOD1_MASK))
if (! press && (tool->button_press_state & GDK_MOD1_MASK))
{
tool->button_press_state &= ~GDK_MOD1_MASK;
}
else
{
gimp_tool_active_modifier_key (tool, GDK_MOD1_MASK,
(state & GDK_MOD1_MASK) ? TRUE : FALSE,
state,
press, state,
display);
}
}