reordered and renamed parameters to make more sense.

2008-01-07  Michael Natterer  <mitch@gimp.org>

	* app/core/gimplayer.c (gimp_layer_transform_color): reordered
	and renamed parameters to make more sense.


svn path=/trunk/; revision=24562
This commit is contained in:
Michael Natterer 2008-01-07 18:53:55 +00:00 committed by Michael Natterer
parent bf02ecb9cf
commit 147f3417fd
2 changed files with 39 additions and 35 deletions

View file

@ -1,3 +1,8 @@
2008-01-07 Michael Natterer <mitch@gimp.org>
* app/core/gimplayer.c (gimp_layer_transform_color): reordered
and renamed parameters to make more sense.
2008-01-07 Michael Natterer <mitch@gimp.org> 2008-01-07 Michael Natterer <mitch@gimp.org>
* app/gegl/Makefile.am (INCLUDES): remove GTK_CFLAGS. * app/gegl/Makefile.am (INCLUDES): remove GTK_CFLAGS.

View file

@ -156,10 +156,10 @@ static gint gimp_layer_get_opacity_at (GimpPickable *pickable,
gint y); gint y);
static void gimp_layer_transform_color (GimpImage *image, static void gimp_layer_transform_color (GimpImage *image,
PixelRegion *layerPR, PixelRegion *srcPR,
PixelRegion *bufPR, GimpImageType src_type,
GimpImageType dest_type, PixelRegion *destPR,
GimpImageType src_type); GimpImageType dest_type);
static void gimp_layer_layer_mask_update (GimpDrawable *layer_mask, static void gimp_layer_layer_mask_update (GimpDrawable *layer_mask,
gint x, gint x,
@ -570,7 +570,7 @@ gimp_layer_convert (GimpItem *item,
if (gimp_drawable_has_alpha (drawable)) if (gimp_drawable_has_alpha (drawable))
new_type = GIMP_IMAGE_TYPE_WITH_ALPHA (new_type); new_type = GIMP_IMAGE_TYPE_WITH_ALPHA (new_type);
new_tiles = tile_manager_new (gimp_item_width (item), new_tiles = tile_manager_new (gimp_item_width (item),
gimp_item_height (item), gimp_item_height (item),
GIMP_IMAGE_TYPE_BYTES (new_type)); GIMP_IMAGE_TYPE_BYTES (new_type));
@ -595,19 +595,18 @@ gimp_layer_convert (GimpItem *item,
pixel_region_init (&layerPR, gimp_drawable_get_tiles (drawable), pixel_region_init (&layerPR, gimp_drawable_get_tiles (drawable),
0, 0, 0, 0,
gimp_item_width (item), gimp_item_width (item),
gimp_item_height (item), gimp_item_height (item),
FALSE); FALSE);
pixel_region_init (&newPR, new_tiles, pixel_region_init (&newPR, new_tiles,
0, 0, 0, 0,
gimp_item_width (item), gimp_item_width (item),
gimp_item_height (item), gimp_item_height (item),
TRUE); TRUE);
gimp_layer_transform_color (dest_image, gimp_layer_transform_color (dest_image,
&newPR, &layerPR, &layerPR, gimp_drawable_type (drawable),
new_type, &newPR, new_type);
gimp_drawable_type (drawable));
} }
break; break;
} }
@ -912,43 +911,43 @@ gimp_layer_get_opacity_at (GimpPickable *pickable,
static void static void
gimp_layer_transform_color (GimpImage *image, gimp_layer_transform_color (GimpImage *image,
PixelRegion *layerPR, PixelRegion *srcPR,
PixelRegion *bufPR, GimpImageType src_type,
GimpImageType dest_type, PixelRegion *destPR,
GimpImageType src_type) GimpImageType dest_type)
{ {
GimpImageBaseType base_type = GIMP_IMAGE_TYPE_BASE_TYPE (src_type); GimpImageBaseType base_type = GIMP_IMAGE_TYPE_BASE_TYPE (src_type);
gboolean alpha = GIMP_IMAGE_TYPE_HAS_ALPHA (src_type); gboolean alpha = GIMP_IMAGE_TYPE_HAS_ALPHA (src_type);
gpointer pr; gpointer pr;
for (pr = pixel_regions_register (2, layerPR, bufPR); for (pr = pixel_regions_register (2, srcPR, destPR);
pr != NULL; pr != NULL;
pr = pixel_regions_process (pr)) pr = pixel_regions_process (pr))
{ {
const guchar *src = bufPR->data; const guchar *src = srcPR->data;
guchar *dest = layerPR->data; guchar *dest = destPR->data;
gint h = layerPR->h; gint h = destPR->h;
while (h--) while (h--)
{ {
const guchar *s = src; const guchar *s = src;
guchar *d = dest; guchar *d = dest;
gint i; gint i;
for (i = 0; i < layerPR->w; i++) for (i = 0; i < destPR->w; i++)
{ {
gimp_image_transform_color (image, dest_type, d, base_type, s); gimp_image_transform_color (image, dest_type, d, base_type, s);
/* alpha channel */ /* alpha channel */
d[layerPR->bytes - 1] = (alpha ? d[destPR->bytes - 1] = (alpha ?
s[bufPR->bytes - 1] : OPAQUE_OPACITY); s[srcPR->bytes - 1] : OPAQUE_OPACITY);
s += bufPR->bytes; s += srcPR->bytes;
d += layerPR->bytes; d += destPR->bytes;
} }
src += bufPR->rowstride; src += srcPR->rowstride;
dest += layerPR->rowstride; dest += destPR->rowstride;
} }
} }
} }
@ -1170,8 +1169,9 @@ gimp_layer_new_from_region (PixelRegion *region,
break; break;
case GIMP_GRAY_IMAGE: case GIMP_GRAY_IMAGE:
case GIMP_GRAYA_IMAGE: case GIMP_GRAYA_IMAGE:
gimp_layer_transform_color (dest_image, &layerPR, region, gimp_layer_transform_color (dest_image,
type, src_type); region, src_type,
&layerPR, type);
break; break;
default: default:
g_warning ("%s: unhandled type conversion", G_STRFUNC); g_warning ("%s: unhandled type conversion", G_STRFUNC);
@ -1196,8 +1196,9 @@ gimp_layer_new_from_region (PixelRegion *region,
{ {
case GIMP_RGB_IMAGE: case GIMP_RGB_IMAGE:
case GIMP_RGBA_IMAGE: case GIMP_RGBA_IMAGE:
gimp_layer_transform_color (dest_image, &layerPR, region, gimp_layer_transform_color (dest_image,
type, src_type); region, src_type,
&layerPR, type);
break; break;
case GIMP_GRAYA_IMAGE: case GIMP_GRAYA_IMAGE:
copy_region (region, &layerPR); copy_region (region, &layerPR);
@ -1220,13 +1221,11 @@ gimp_layer_new_from_region (PixelRegion *region,
{ {
case GIMP_RGB_IMAGE: case GIMP_RGB_IMAGE:
case GIMP_RGBA_IMAGE: case GIMP_RGBA_IMAGE:
gimp_layer_transform_color (dest_image, &layerPR, region,
type, src_type);
break;
case GIMP_GRAY_IMAGE: case GIMP_GRAY_IMAGE:
case GIMP_GRAYA_IMAGE: case GIMP_GRAYA_IMAGE:
gimp_layer_transform_color (dest_image, &layerPR, region, gimp_layer_transform_color (dest_image,
type, src_type); region, src_type,
&layerPR, type);
break; break;
default: default:
g_warning ("%s: unhandled type conversion", G_STRFUNC); g_warning ("%s: unhandled type conversion", G_STRFUNC);