[gimp/soc-2010-cage-2] refactor of the GimpCageConfig with a custom GimpCagePoint data structure
- From: Michael Muré <mmure src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gimp/soc-2010-cage-2] refactor of the GimpCageConfig with a custom GimpCagePoint data structure
- Date: Thu, 30 Dec 2010 23:50:56 +0000 (UTC)
commit 2785911a8cf9cfed8d37f8dc6efa919a58fb5aaa
Author: Michael Muré <batolettre gmail com>
Date: Fri Dec 31 00:50:08 2010 +0100
refactor of the GimpCageConfig with a custom GimpCagePoint data structure
app/core/core-types.h | 12 +++-
app/gegl/gimpcageconfig.c | 156 +++++++++++++--------------------
app/gegl/gimpcageconfig.h | 45 ++++-----
app/gegl/gimpoperationcagecoefcalc.c | 4 +-
app/gegl/gimpoperationcagetransform.c | 12 ++--
app/tools/gimpcagetool.c | 63 ++++++++-----
app/tools/gimpcagetool.h | 2 +-
7 files changed, 139 insertions(+), 155 deletions(-)
---
diff --git a/app/core/core-types.h b/app/core/core-types.h
index ace5370..e7e360c 100644
--- a/app/core/core-types.h
+++ b/app/core/core-types.h
@@ -21,6 +21,8 @@
#include "libgimpmodule/gimpmoduletypes.h"
#include "libgimpthumb/gimpthumb-types.h"
+#include "libgimpmath/gimpmathtypes.h"
+#include "libgimpmath/gimpvector.h"
#include "base/base-types.h"
@@ -174,6 +176,7 @@ typedef struct _GimpTagged GimpTagged; /* dummy typedef */
/* non-object types */
typedef struct _GimpArea GimpArea;
+typedef struct _GimpCagePoint GimpCagePoint;
typedef struct _GimpCoords GimpCoords;
typedef struct _GimpGradientSegment GimpGradientSegment;
typedef struct _GimpPaletteEntry GimpPaletteEntry;
@@ -201,6 +204,14 @@ typedef void (* GimpImageMapApplyFunc) (gpointer apply_data,
/* structs */
+struct _GimpCagePoint
+{
+ GimpVector2 src_point;
+ GimpVector2 dest_point;
+ GimpVector2 edge_normal;
+ gdouble edge_scaling_factor;
+};
+
struct _GimpCoords
{
gdouble x;
@@ -213,7 +224,6 @@ struct _GimpCoords
gdouble direction;
};
-
#include "gegl/gimp-gegl-types.h"
#include "paint/paint-types.h"
#include "text/text-types.h"
diff --git a/app/gegl/gimpcageconfig.c b/app/gegl/gimpcageconfig.c
index 2df9c15..6d7458a 100644
--- a/app/gegl/gimpcageconfig.c
+++ b/app/gegl/gimpcageconfig.c
@@ -33,6 +33,9 @@
/*#define DEBUG_CAGE */
#define N_ITEMS_PER_ALLOC 10
+/* This DELTA is aimed to not have handle on exact pixel during computation,
+ * to avoid particular case. It shouldn't be so usefull, but it's a double
+ * safety. */
#define DELTA 0.010309278351
@@ -94,10 +97,7 @@ gimp_cage_config_init (GimpCageConfig *self)
self->n_cage_vertices = 0;
self->max_cage_vertices = 50; /*pre-allocation for 50 vertices for the cage.*/
- self->cage_vertices = g_new0 (GimpVector2, self->max_cage_vertices);
- self->cage_vertices_d = g_new0 (GimpVector2, self->max_cage_vertices);
- self->scaling_factor = g_malloc0 (self->max_cage_vertices * sizeof (gdouble));
- self->normal_d = g_new0 (GimpVector2, self->max_cage_vertices);
+ self->cage_points = g_new0 (GimpCagePoint, self->max_cage_vertices);
}
static void
@@ -105,10 +105,7 @@ gimp_cage_config_finalize (GObject *object)
{
GimpCageConfig *gcc = GIMP_CAGE_CONFIG (object);
- g_free (gcc->cage_vertices);
- g_free (gcc->cage_vertices_d);
- g_free (gcc->scaling_factor);
- g_free (gcc->normal_d);
+ g_free (gcc->cage_points);
G_OBJECT_CLASS (parent_class)->finalize (object);
}
@@ -162,27 +159,16 @@ gimp_cage_config_add_cage_point (GimpCageConfig *gcc,
{
gcc->max_cage_vertices += N_ITEMS_PER_ALLOC;
- gcc->cage_vertices = g_renew (GimpVector2,
- gcc->cage_vertices,
- gcc->max_cage_vertices);
-
- gcc->cage_vertices_d = g_renew (GimpVector2,
- gcc->cage_vertices_d,
- gcc->max_cage_vertices);
-
- gcc->scaling_factor = g_realloc (gcc->scaling_factor,
- gcc->max_cage_vertices * sizeof (gdouble));
-
- gcc->normal_d = g_renew (GimpVector2,
- gcc->normal_d,
- gcc->max_cage_vertices);
+ gcc->cage_points = g_renew (GimpCagePoint,
+ gcc->cage_points,
+ gcc->max_cage_vertices);
}
- gcc->cage_vertices[gcc->n_cage_vertices].x = x + DELTA - gcc->offset_x;
- gcc->cage_vertices[gcc->n_cage_vertices].y = y + DELTA - gcc->offset_y;
+ gcc->cage_points[gcc->n_cage_vertices].src_point.x = x + DELTA - gcc->offset_x;
+ gcc->cage_points[gcc->n_cage_vertices].src_point.y = y + DELTA - gcc->offset_y;
- gcc->cage_vertices_d[gcc->n_cage_vertices].x = x + DELTA - gcc->offset_x;
- gcc->cage_vertices_d[gcc->n_cage_vertices].y = y + DELTA - gcc->offset_y;
+ gcc->cage_points[gcc->n_cage_vertices].dest_point.x = x + DELTA - gcc->offset_x;
+ gcc->cage_points[gcc->n_cage_vertices].dest_point.y = y + DELTA - gcc->offset_y;
gcc->n_cage_vertices++;
@@ -232,15 +218,15 @@ gimp_cage_config_move_cage_point (GimpCageConfig *gcc,
if (mode == GIMP_CAGE_MODE_CAGE_CHANGE)
{
- gcc->cage_vertices[point_number].x = x + DELTA - gcc->offset_x;
- gcc->cage_vertices[point_number].y = y + DELTA - gcc->offset_y;
- gcc->cage_vertices_d[point_number].x = x + DELTA - gcc->offset_x;
- gcc->cage_vertices_d[point_number].y = y + DELTA - gcc->offset_y;
+ gcc->cage_points[point_number].src_point.x = x + DELTA - gcc->offset_x;
+ gcc->cage_points[point_number].src_point.y = y + DELTA - gcc->offset_y;
+ gcc->cage_points[point_number].dest_point.x = x + DELTA - gcc->offset_x;
+ gcc->cage_points[point_number].dest_point.y = y + DELTA - gcc->offset_y;
}
else
{
- gcc->cage_vertices_d[point_number].x = x + DELTA - gcc->offset_x;
- gcc->cage_vertices_d[point_number].y = y + DELTA - gcc->offset_y;
+ gcc->cage_points[point_number].dest_point.x = x + DELTA - gcc->offset_x;
+ gcc->cage_points[point_number].dest_point.y = y + DELTA - gcc->offset_y;
}
gimp_cage_config_compute_scaling_factor (gcc);
@@ -264,8 +250,8 @@ gimp_cage_config_get_bounding_box (GimpCageConfig *gcc)
g_return_val_if_fail (GIMP_IS_CAGE_CONFIG (gcc), bounding_box);
g_return_val_if_fail (gcc->n_cage_vertices >= 0, bounding_box);
- bounding_box.x = gcc->cage_vertices[0].x;
- bounding_box.y = gcc->cage_vertices[0].y;
+ bounding_box.x = gcc->cage_points[0].src_point.x;
+ bounding_box.y = gcc->cage_points[0].src_point.y;
bounding_box.height = 0;
bounding_box.width = 0;
@@ -273,8 +259,8 @@ gimp_cage_config_get_bounding_box (GimpCageConfig *gcc)
{
gdouble x,y;
- x = gcc->cage_vertices[i].x;
- y = gcc->cage_vertices[i].y;
+ x = gcc->cage_points[i].src_point.x;
+ y = gcc->cage_points[i].src_point.y;
if (x < bounding_box.x)
{
@@ -313,20 +299,16 @@ gimp_cage_config_get_bounding_box (GimpCageConfig *gcc)
void
gimp_cage_config_reverse_cage (GimpCageConfig *gcc)
{
- GimpVector2 temp;
+ GimpCagePoint temp;
gint i;
g_return_if_fail (GIMP_IS_CAGE_CONFIG (gcc));
for (i = 0; i < gcc->n_cage_vertices / 2; i++)
{
- temp = gcc->cage_vertices[i];
- gcc->cage_vertices[i] = gcc->cage_vertices[gcc->n_cage_vertices - i - 1];
- gcc->cage_vertices[gcc->n_cage_vertices - i - 1] = temp;
-
- temp = gcc->cage_vertices_d[i];
- gcc->cage_vertices_d[i] = gcc->cage_vertices_d[gcc->n_cage_vertices - i - 1];
- gcc->cage_vertices_d[gcc->n_cage_vertices - i - 1] = temp;
+ temp = gcc->cage_points[i];
+ gcc->cage_points[i] = gcc->cage_points[gcc->n_cage_vertices - i - 1];
+ gcc->cage_points[gcc->n_cage_vertices - i - 1] = temp;
}
gimp_cage_config_compute_scaling_factor (gcc);
@@ -360,9 +342,9 @@ gimp_cage_config_reverse_cage_if_needed (GimpCageConfig *gcc)
GimpVector2 P1, P2, P3;
gdouble z;
- P1 = gcc->cage_vertices[i];
- P2 = gcc->cage_vertices[(i+1) % gcc->n_cage_vertices];
- P3 = gcc->cage_vertices[(i+2) % gcc->n_cage_vertices];
+ P1 = gcc->cage_points[i].src_point;
+ P2 = gcc->cage_points[(i+1) % gcc->n_cage_vertices].src_point;
+ P3 = gcc->cage_points[(i+2) % gcc->n_cage_vertices].src_point;
z = P1.x * (P2.y - P3.y) + P2.x * (P3.y - P1.y) + P3.x * (P1.y - P2.y);
@@ -377,67 +359,48 @@ gimp_cage_config_reverse_cage_if_needed (GimpCageConfig *gcc)
}
/**
- * gimp_cage_config_get_cage_point:
+ * gimp_cage_config_get_cage_points:
* @gcc: the cage config
- * @mode: the actual mode of the cage, GIMP_CAGE_MODE_CAGE_CHANGE or GIMP_CAGE_MODE_DEFORM
*
- * Returns: a copy of the cage's point, for the cage mode provided
+ * Returns: a copy of the cage's point
*/
-GimpVector2*
-gimp_cage_config_get_cage_point (GimpCageConfig *gcc,
- GimpCageMode mode)
+GimpCagePoint*
+gimp_cage_config_get_cage_points (GimpCageConfig *gcc)
{
- gint x;
- GimpVector2 *vertices_cage;
- GimpVector2 *vertices_copy;
+ gint i;
+ GimpCagePoint *points_copy;
g_return_val_if_fail (GIMP_IS_CAGE_CONFIG (gcc), NULL);
- vertices_copy = g_new (GimpVector2, gcc->n_cage_vertices);
+ points_copy = g_new (GimpCagePoint, gcc->n_cage_vertices);
- if (mode == GIMP_CAGE_MODE_CAGE_CHANGE)
- vertices_cage = gcc->cage_vertices;
- else
- vertices_cage = gcc->cage_vertices_d;
-
- for(x = 0; x < gcc->n_cage_vertices; x++)
+ for(i = 0; i < gcc->n_cage_vertices; i++)
{
- vertices_copy[x] = vertices_cage[x];
+ points_copy[i] = gcc->cage_points[i];
}
- return vertices_copy;
+ return points_copy;
}
/**
- * gimp_cage_config_commit_cage_point:
+ * gimp_cage_config_commit_cage_points:
* @gcc: the cage config
- * @mode: the actual mode of the cage, GIMP_CAGE_MODE_CAGE_CHANGE or GIMP_CAGE_MODE_DEFORM
- * @points: a GimpVector2 array of point of the same length as the number of point in the cage
+ * @points: a GimpCagePoint array of point of the same length as the number of point in the cage
*
* This function update the cage's point with the array provided.
*/
void
-gimp_cage_config_commit_cage_point (GimpCageConfig *gcc,
- GimpCageMode mode,
- GimpVector2 *points)
+gimp_cage_config_commit_cage_points (GimpCageConfig *gcc,
+ GimpCagePoint *points)
{
gint i;
- GimpVector2 *vertices;
g_return_if_fail (GIMP_IS_CAGE_CONFIG (gcc));
- if (mode == GIMP_CAGE_MODE_CAGE_CHANGE)
- vertices = gcc->cage_vertices;
- else
- vertices = gcc->cage_vertices_d;
-
for(i = 0; i < gcc->n_cage_vertices; i++)
{
- vertices[i] = points[i];
+ gcc->cage_points[i] = points[i];
}
-
- gimp_cage_config_compute_scaling_factor (gcc);
- gimp_cage_config_compute_edge_normal (gcc);
}
static void
@@ -452,16 +415,16 @@ gimp_cage_config_compute_scaling_factor (GimpCageConfig *gcc)
for (i = 0; i < gcc->n_cage_vertices; i++)
{
gimp_vector2_sub (&edge,
- &gcc->cage_vertices[i],
- &gcc->cage_vertices[(i + 1) % gcc->n_cage_vertices]);
+ &gcc->cage_points[i].src_point,
+ &gcc->cage_points[(i + 1) % gcc->n_cage_vertices].src_point);
length = gimp_vector2_length (&edge);
gimp_vector2_sub (&edge,
- &gcc->cage_vertices_d[i],
- &gcc->cage_vertices_d[(i + 1) % gcc->n_cage_vertices]);
+ &gcc->cage_points[i].dest_point,
+ &gcc->cage_points[(i + 1) % gcc->n_cage_vertices].dest_point);
length_d = gimp_vector2_length (&edge);
- gcc->scaling_factor[i] = length_d / length;
+ gcc->cage_points[i].edge_scaling_factor = length_d / length;
}
#ifdef DEBUG_CAGE
@@ -480,10 +443,10 @@ gimp_cage_config_compute_edge_normal (GimpCageConfig *gcc)
for (i = 0; i < gcc->n_cage_vertices; i++)
{
gimp_vector2_sub (&normal,
- &gcc->cage_vertices_d[(i + 1) % gcc->n_cage_vertices],
- &gcc->cage_vertices_d[i]);
+ &gcc->cage_points[(i + 1) % gcc->n_cage_vertices].dest_point,
+ &gcc->cage_points[i].dest_point);
- gcc->normal_d[i] = gimp_vector2_normal (&normal);
+ gcc->cage_points[i].edge_normal = gimp_vector2_normal (&normal);
}
}
@@ -494,7 +457,7 @@ gimp_cage_config_compute_edge_normal (GimpCageConfig *gcc)
* @y: y coordinate of the point to test
*
* Check if the given point is inside the cage. This test is done in
- * the regard of the topological inside of the cage.
+ * the regard of the topological inside of the source cage.
*
* Returns: TRUE if the point is inside, FALSE if not.
*/
@@ -503,21 +466,22 @@ gimp_cage_config_point_inside (GimpCageConfig *gcc,
gfloat x,
gfloat y)
{
- GimpVector2 *cv;
+ GimpVector2 *cpi, *cpj;
gboolean inside = FALSE;
gint i, j;
g_return_val_if_fail (GIMP_IS_CAGE_CONFIG (gcc), FALSE);
- cv = gcc->cage_vertices;
-
for (i = 0, j = gcc->n_cage_vertices - 1;
i < gcc->n_cage_vertices;
j = i++)
{
- if ((((cv[i].y <= y) && (y < cv[j].y))
- || ((cv[j].y <= y) && (y < cv[i].y)))
- && (x < (cv[j].x - cv[i].x) * (y - cv[i].y) / (cv[j].y - cv[i].y) + cv[i].x))
+ cpi = &(gcc->cage_points[i].src_point);
+ cpj = &(gcc->cage_points[j].src_point);
+
+ if ((((cpi->y <= y) && (y < cpj->y))
+ || ((cpj->y <= y) && (y < cpi->y)))
+ && (x < (cpj->x - cpi->x) * (y - cpi->y) / (cpj->y - cpi->y) + cpi->x))
{
inside = !inside;
}
diff --git a/app/gegl/gimpcageconfig.h b/app/gegl/gimpcageconfig.h
index c689fa9..3e6841b 100644
--- a/app/gegl/gimpcageconfig.h
+++ b/app/gegl/gimpcageconfig.h
@@ -44,10 +44,7 @@ struct _GimpCageConfig
gint offset_x;
gint offset_y;
- GimpVector2 *cage_vertices; /* cage before deformation */
- GimpVector2 *cage_vertices_d; /* cage after deformation */
- gdouble *scaling_factor;
- GimpVector2 *normal_d;
+ GimpCagePoint *cage_points;
};
struct _GimpCageConfigClass
@@ -55,26 +52,24 @@ struct _GimpCageConfigClass
GimpImageMapConfigClass parent_class;
};
-GType gimp_cage_config_get_type (void) G_GNUC_CONST;
+GType gimp_cage_config_get_type (void) G_GNUC_CONST;
-void gimp_cage_config_add_cage_point (GimpCageConfig *gcc,
- gdouble x,
- gdouble y);
-void gimp_cage_config_remove_last_cage_point (GimpCageConfig *gcc);
-void gimp_cage_config_move_cage_point (GimpCageConfig *gcc,
- GimpCageMode mode,
- gint point_number,
- gdouble x,
- gdouble y);
-GeglRectangle gimp_cage_config_get_bounding_box (GimpCageConfig *gcc);
-void gimp_cage_config_reverse_cage_if_needed (GimpCageConfig *gcc);
-void gimp_cage_config_reverse_cage (GimpCageConfig *gcc);
-gboolean gimp_cage_config_point_inside (GimpCageConfig *gcc,
- gfloat x,
- gfloat y);
-GimpVector2* gimp_cage_config_get_cage_point (GimpCageConfig *gcc,
- GimpCageMode mode);
-void gimp_cage_config_commit_cage_point (GimpCageConfig *gcc,
- GimpCageMode mode,
- GimpVector2 *points);
+void gimp_cage_config_add_cage_point (GimpCageConfig *gcc,
+ gdouble x,
+ gdouble y);
+void gimp_cage_config_remove_last_cage_point (GimpCageConfig *gcc);
+void gimp_cage_config_move_cage_point (GimpCageConfig *gcc,
+ GimpCageMode mode,
+ gint point_number,
+ gdouble x,
+ gdouble y);
+GeglRectangle gimp_cage_config_get_bounding_box (GimpCageConfig *gcc);
+void gimp_cage_config_reverse_cage_if_needed (GimpCageConfig *gcc);
+void gimp_cage_config_reverse_cage (GimpCageConfig *gcc);
+gboolean gimp_cage_config_point_inside (GimpCageConfig *gcc,
+ gfloat x,
+ gfloat y);
+GimpCagePoint* gimp_cage_config_get_cage_points (GimpCageConfig *gcc);
+void gimp_cage_config_commit_cage_points (GimpCageConfig *gcc,
+ GimpCagePoint *points);
#endif /* __GIMP_CAGE_CONFIG_H__ */
diff --git a/app/gegl/gimpoperationcagecoefcalc.c b/app/gegl/gimpoperationcagecoefcalc.c
index 55dae56..5455577 100644
--- a/app/gegl/gimpoperationcagecoefcalc.c
+++ b/app/gegl/gimpoperationcagecoefcalc.c
@@ -221,8 +221,8 @@ gimp_operation_cage_coef_calc_process (GeglOperation *operation,
GimpVector2 v1,v2,a,b,p;
gdouble BA,SRT,L0,L1,A0,A1,A10,L10, Q,S,R, absa;
- v1 = config->cage_vertices[j];
- v2 = config->cage_vertices[(j+1)%config->n_cage_vertices];
+ v1 = config->cage_points[j].src_point;
+ v2 = config->cage_points[(j+1)%config->n_cage_vertices].src_point;
p.x = x;
p.y = y;
a.x = v2.x - v1.x;
diff --git a/app/gegl/gimpoperationcagetransform.c b/app/gegl/gimpoperationcagetransform.c
index 47ece5f..2bcff13 100644
--- a/app/gegl/gimpoperationcagetransform.c
+++ b/app/gegl/gimpoperationcagetransform.c
@@ -220,8 +220,8 @@ gimp_operation_cage_transform_process (GeglOperation *operation,
/* pre-fill the out buffer with no-displacement coordinate */
it = gegl_buffer_iterator_new (out_buf, roi, NULL, GEGL_BUFFER_WRITE);
- plain_color.x = (gint) config->cage_vertices[0].x;
- plain_color.y = (gint) config->cage_vertices[0].y;
+ plain_color.x = (gint) config->cage_points[0].src_point.x;
+ plain_color.y = (gint) config->cage_points[0].src_point.y;
while (gegl_buffer_iterator_next (it))
{
@@ -507,14 +507,14 @@ gimp_cage_transform_compute_destination (GimpCageConfig *config,
for (i = 0; i < cvn; i++)
{
- pos_x += coef[i] * config->cage_vertices_d[i].x;
- pos_y += coef[i] * config->cage_vertices_d[i].y;
+ pos_x += coef[i] * config->cage_points[i].dest_point.x;
+ pos_y += coef[i] * config->cage_points[i].dest_point.y;
}
for (i = 0; i < cvn; i++)
{
- pos_x += coef[i + cvn] * config->scaling_factor[i] * config->normal_d[i].x;
- pos_y += coef[i + cvn] * config->scaling_factor[i] * config->normal_d[i].y;
+ pos_x += coef[i + cvn] * config->cage_points[i].edge_scaling_factor * config->cage_points[i].edge_normal.x;
+ pos_y += coef[i + cvn] * config->cage_points[i].edge_scaling_factor * config->cage_points[i].edge_normal.y;
}
result.x = pos_x;
diff --git a/app/tools/gimpcagetool.c b/app/tools/gimpcagetool.c
index 6350377..0ce4829 100644
--- a/app/tools/gimpcagetool.c
+++ b/app/tools/gimpcagetool.c
@@ -456,7 +456,7 @@ gimp_cage_tool_button_press (GimpTool *tool,
ct->cage_backup = NULL;
}
- ct->cage_backup = gimp_cage_config_get_cage_point (ct->config, options->cage_mode);
+ ct->cage_backup = gimp_cage_config_get_cage_points (ct->config);
if (handle >= 0)
/* User clicked on a handle, so we move it */
@@ -477,7 +477,6 @@ gimp_cage_tool_button_release (GimpTool *tool,
GimpDisplay *display)
{
GimpCageTool *ct = GIMP_CAGE_TOOL (tool);
- GimpCageOptions *options = GIMP_CAGE_TOOL_GET_OPTIONS (ct);
gimp_draw_tool_pause (GIMP_DRAW_TOOL (ct));
@@ -495,9 +494,8 @@ gimp_cage_tool_button_release (GimpTool *tool,
break;
case DEFORM_STATE_MOVE_HANDLE:
- gimp_cage_config_commit_cage_point (ct->config,
- options->cage_mode,
- ct->cage_backup);
+ gimp_cage_config_commit_cage_points (ct->config,
+ ct->cage_backup);
gimp_cage_tool_image_map_update (ct);
ct->tool_state = DEFORM_STATE_WAIT;
break;
@@ -565,7 +563,6 @@ gimp_cage_tool_draw (GimpDrawTool *draw_tool)
GimpCageOptions *options = GIMP_CAGE_TOOL_GET_OPTIONS (ct);
GimpCageConfig *config = ct->config;
GimpCanvasGroup *stroke_group;
- GimpVector2 *vertices;
gint n_vertices;
gint i;
GimpHandleType handle;
@@ -577,21 +574,23 @@ gimp_cage_tool_draw (GimpDrawTool *draw_tool)
stroke_group = gimp_draw_tool_add_stroke_group (draw_tool);
- if (options->cage_mode == GIMP_CAGE_MODE_CAGE_CHANGE)
- vertices = config->cage_vertices;
- else
- vertices = config->cage_vertices_d;
-
gimp_draw_tool_push_group (draw_tool, stroke_group);
/* If needed, draw ligne to the cursor. */
if (ct->tool_state == CAGE_STATE_WAIT || ct->tool_state == CAGE_STATE_MOVE_HANDLE)
{
- gimp_draw_tool_add_line (draw_tool,
- vertices[n_vertices - 1].x + ct->config->offset_x,
- vertices[n_vertices - 1].y + ct->config->offset_y,
- ct->cursor_x,
- ct->cursor_y);
+ if (options->cage_mode == GIMP_CAGE_MODE_CAGE_CHANGE)
+ gimp_draw_tool_add_line (draw_tool,
+ config->cage_points[n_vertices - 1].src_point.x + ct->config->offset_x,
+ config->cage_points[n_vertices - 1].src_point.y + ct->config->offset_y,
+ ct->cursor_x,
+ ct->cursor_y);
+ else
+ gimp_draw_tool_add_line (draw_tool,
+ config->cage_points[n_vertices - 1].dest_point.x + ct->config->offset_x,
+ config->cage_points[n_vertices - 1].dest_point.y + ct->config->offset_y,
+ ct->cursor_x,
+ ct->cursor_y);
}
gimp_draw_tool_pop_group (draw_tool);
@@ -601,8 +600,16 @@ gimp_cage_tool_draw (GimpDrawTool *draw_tool)
{
gdouble x1, y1;
- x1 = vertices[i].x;
- y1 = vertices[i].y;
+ if (options->cage_mode == GIMP_CAGE_MODE_CAGE_CHANGE)
+ {
+ x1 = config->cage_points[i].src_point.x;
+ y1 = config->cage_points[i].src_point.y;
+ }
+ else
+ {
+ x1 = config->cage_points[i].dest_point.x;
+ y1 = config->cage_points[i].dest_point.y;
+ }
if (i > 0 || ct->cage_complete)
{
@@ -614,8 +621,16 @@ gimp_cage_tool_draw (GimpDrawTool *draw_tool)
else
point2 = i - 1;
- x2 = vertices[point2].x;
- y2 = vertices[point2].y;
+ if (options->cage_mode == GIMP_CAGE_MODE_CAGE_CHANGE)
+ {
+ x2 = config->cage_points[point2].src_point.x;
+ y2 = config->cage_points[point2].src_point.y;
+ }
+ else
+ {
+ x2 = config->cage_points[point2].dest_point.x;
+ y2 = config->cage_points[point2].dest_point.y;
+ }
gimp_draw_tool_push_group (draw_tool, stroke_group);
@@ -666,13 +681,13 @@ gimp_cage_tool_is_on_handle (GimpCageConfig *gcc,
{
if (mode == GIMP_CAGE_MODE_CAGE_CHANGE)
{
- vert_x = gcc->cage_vertices[i].x + gcc->offset_x;
- vert_y = gcc->cage_vertices[i].y + gcc->offset_y;
+ vert_x = gcc->cage_points[i].src_point.x + gcc->offset_x;
+ vert_y = gcc->cage_points[i].src_point.y + gcc->offset_y;
}
else
{
- vert_x = gcc->cage_vertices_d[i].x + gcc->offset_x;
- vert_y = gcc->cage_vertices_d[i].y + gcc->offset_y;
+ vert_x = gcc->cage_points[i].dest_point.x + gcc->offset_x;
+ vert_y = gcc->cage_points[i].dest_point.y + gcc->offset_y;
}
dist = gimp_draw_tool_calc_distance_square (GIMP_DRAW_TOOL (draw_tool),
diff --git a/app/tools/gimpcagetool.h b/app/tools/gimpcagetool.h
index 99279cb..144daf6 100644
--- a/app/tools/gimpcagetool.h
+++ b/app/tools/gimpcagetool.h
@@ -56,7 +56,7 @@ struct _GimpCageTool
GimpImageMap *image_map;
- GimpVector2 *cage_backup;
+ GimpCagePoint *cage_backup;
};
struct _GimpCageToolClass
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
