% July 1992 %
% %
% %
-% Copyright 1999-2012 ImageMagick Studio LLC, a non-profit organization %
+% Copyright 1999-2013 ImageMagick Studio LLC, a non-profit organization %
% dedicated to making software imaging solutions freely available. %
% %
% You may not use this file except in compliance with the License. You may %
exception);
(void) QueryColorCompliance(TransparentColor,AllCompliance,
&image->transparent_color,exception);
- image->resolution.x=DefaultResolution;
- image->resolution.y=DefaultResolution;
- image->units=PixelsPerInchResolution;
GetTimerInfo(&image->timer);
image->cache=AcquirePixelCache(0);
image->channel_mask=DefaultChannels;
image->channel_map=AcquirePixelChannelMap();
image->blob=CloneBlobInfo((BlobInfo *) NULL);
+ image->timestamp=time((time_t *) NULL);
image->debug=IsEventLogging();
image->reference_count=1;
image->semaphore=AllocateSemaphoreInfo();
image->client_data=image_info->client_data;
if (image_info->cache != (void *) NULL)
ClonePixelCacheMethods(image->cache,image_info->cache);
+ /*
+ Set all global options that map to per-image settings.
+ */
(void) SyncImageSettings(image_info,image,exception);
+ /*
+ Global options that are only set for new images.
+ */
+ image->image_info=(ImageInfo *) NULL;
option=GetImageOption(image_info,"delay");
if (option != (const char *) NULL)
{
*append_image;
MagickBooleanType
- matte,
- proceed,
status;
MagickOffsetType
n;
+ PixelTrait
+ alpha_trait;
+
RectangleInfo
geometry;
(void) LogMagickEvent(TraceEvent,GetMagickModule(),"%s",images->filename);
assert(exception != (ExceptionInfo *) NULL);
assert(exception->signature == MagickSignature);
- matte=images->matte;
+ alpha_trait=images->alpha_trait;
number_images=1;
width=images->columns;
height=images->rows;
next=GetNextImageInList(images);
for ( ; next != (Image *) NULL; next=GetNextImageInList(next))
{
- if (next->matte != MagickFalse)
- matte=MagickTrue;
+ if (next->alpha_trait == BlendPixelTrait)
+ alpha_trait=BlendPixelTrait;
number_images++;
if (stack != MagickFalse)
{
append_image=DestroyImage(append_image);
return((Image *) NULL);
}
- append_image->matte=matte;
+ append_image->alpha_trait=alpha_trait;
(void) SetImageBackgroundColor(append_image,exception);
status=MagickTrue;
x_offset=0;
Image
*image;
+ MagickBooleanType
+ proceed;
+
image=CloneImage(next,0,0,MagickTrue,exception);
if (image == (Image *) NULL)
break;
y_offset-=geometry.y;
image_view=AcquireVirtualCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
- #pragma omp parallel for schedule(static) shared(status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ #pragma omp parallel for schedule(static,4) shared(status) \
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
Image
*clone_image;
- MagickRealType
+ double
scale;
size_t
clone_image->metacontent_extent=image->metacontent_extent;
clone_image->colorspace=image->colorspace;
clone_image->mask=image->mask;
- clone_image->matte=image->matte;
+ clone_image->alpha_trait=image->alpha_trait;
clone_image->columns=image->columns;
clone_image->rows=image->rows;
clone_image->dither=image->dither;
clone_image->cache=ReferencePixelCache(image->cache);
return(clone_image);
}
- scale=(MagickRealType) columns/(MagickRealType) image->columns;
+ scale=(double) columns/(double) image->columns;
clone_image->page.width=(size_t) floor(scale*image->page.width+0.5);
clone_image->page.x=(ssize_t) ceil(scale*image->page.x-0.5);
clone_image->tile_offset.x=(ssize_t) ceil(scale*image->tile_offset.x-0.5);
- scale=(MagickRealType) rows/(MagickRealType) image->rows;
+ scale=(double) rows/(double) image->rows;
clone_image->page.height=(size_t) floor(scale*image->page.height+0.5);
clone_image->page.y=(ssize_t) ceil(scale*image->page.y-0.5);
clone_image->tile_offset.y=(ssize_t) ceil(scale*image->tile_offset.y-0.5);
if (image->debug != MagickFalse)
(void) LogMagickEvent(TraceEvent,GetMagickModule(),"...");
assert(image->signature == MagickSignature);
- return(image->matte);
+ return(image->alpha_trait == BlendPixelTrait ? MagickTrue : MagickFalse);
}
\f
/*
*/
MagickExport void GetImageInfo(ImageInfo *image_info)
{
+ char
+ *synchronize;
+
ExceptionInfo
*exception;
image_info->quality=UndefinedCompressionQuality;
image_info->antialias=MagickTrue;
image_info->dither=MagickTrue;
- image_info->synchronize=IsStringTrue(GetEnvironmentValue(
- "MAGICK_SYNCHRONIZE"));
+ synchronize=GetEnvironmentValue("MAGICK_SYNCHRONIZE");
+ if (synchronize != (const char *) NULL)
+ {
+ image_info->synchronize=IsStringTrue(synchronize);
+ synchronize=DestroyString(synchronize);
+ }
exception=AcquireExceptionInfo();
(void) QueryColorCompliance(BackgroundColor,AllCompliance,
&image_info->background_color,exception);
if (mask_image == (Image *) NULL)
return((Image *) NULL);
status=MagickTrue;
- SetImageColorspace(mask_image,GRAYColorspace,exception);
+ (void) SetImageColorspace(mask_image,GRAYColorspace,exception);
mask_image->mask=MagickFalse;
image_view=AcquireVirtualCacheView(image,exception);
mask_view=AcquireAuthenticCacheView(mask_image,exception);
}
mask_view=DestroyCacheView(mask_view);
image_view=DestroyCacheView(image_view);
+ if (status == MagickFalse)
+ mask_image=DestroyImage(mask_image);
return(mask_image);
}
\f
Image option.
*/
/* FUTURE: Compare update with code from InterpretImageProperties()
- Note that a 'filename:' properity should not need depth recursion.
+ Note that a 'filename:' property should not need depth recursion.
*/
if (strchr(p,']') == (char *) NULL)
break;
#if 0
// FUTURE: remove this code. -- Anthony 29 Arpil 2012
// Removed as GetMagickProperty() will will never match a "filename:"
- // string as this is not a 'known' image properity.
+ // string as this is not a 'known' image property.
//
if ((image_info != (const ImageInfo *) NULL) &&
(image != (const Image *) NULL))
#endif
if (image != (Image *) NULL)
value=GetImageProperty(image,pattern,exception);
- if ((value == (const char *) NULL) &&
- (image != (Image *) NULL))
+ if ((value == (const char *) NULL) && (image != (Image *) NULL))
value=GetImageArtifact(image,pattern);
if ((value == (const char *) NULL) &&
(image_info != (ImageInfo *) NULL))
image_view=AcquireVirtualCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
#pragma omp parallel for schedule(static,4) shared(status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
}
for (x=0; x < (ssize_t) image->columns; x++)
{
- PixelTrait
- traits;
-
register ssize_t
i;
}
for (i=0; i < (ssize_t) GetPixelChannels(image); i++)
{
- MagickRealType
+ double
pixel;
- traits=GetPixelChannelMapTraits(image,i);
+ PixelTrait
+ traits;
+
+ traits=GetPixelChannelTraits(image,(PixelChannel) i);
if (traits == UndefinedPixelTrait)
continue;
- pixel=(MagickRealType) p[i];
+ pixel=(double) p[i];
if ((pixel < 0.0) || (pixel > QuantumRange) ||
(pixel != (QuantumAny) pixel))
break;
image->columns=width;
image->rows=height;
image->colorspace=background->colorspace;
- image->matte=background->matte;
+ image->alpha_trait=background->alpha_trait;
image->fuzz=background->fuzz;
image->depth=background->depth;
status=MagickTrue;
image_view=AcquireAuthenticCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
#pragma omp parallel for schedule(static,4) shared(status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
% The format of the SetImageAlphaChannel method is:
%
% MagickBooleanType SetImageAlphaChannel(Image *image,
-% const AlphaChannelType alpha_type,ExceptionInfo *exception)
+% const AlphaChannelOption alpha_type,ExceptionInfo *exception)
%
% A description of each parameter follows:
%
*/
static inline void FlattenPixelInfo(const Image *image,const PixelInfo *p,
- const MagickRealType alpha,const Quantum *q,const MagickRealType beta,
+ const double alpha,const Quantum *q,const double beta,
Quantum *composite)
{
- MagickRealType
+ double
Da,
gamma,
Sa;
Sa=QuantumScale*alpha;
Da=QuantumScale*beta,
gamma=Sa*(-Da)+Sa+Da;
- gamma=1.0/(gamma <= MagickEpsilon ? 1.0 : gamma);
+ gamma=PerceptibleReciprocal(gamma);
for (i=0; i < (ssize_t) GetPixelChannels(image); i++)
{
- PixelChannel
- channel;
-
- PixelTrait
- traits;
-
- channel=GetPixelChannelMapChannel(image,i);
- traits=GetPixelChannelMapTraits(image,channel);
+ PixelChannel channel=GetPixelChannelChannel(image,i);
+ PixelTrait traits=GetPixelChannelTraits(image,channel);
if (traits == UndefinedPixelTrait)
continue;
switch (channel)
{
case RedPixelChannel:
{
- composite[i]=ClampToQuantum(gamma*MagickOver_((MagickRealType) q[i],
- beta,(MagickRealType) p->red,alpha));
+ composite[i]=ClampToQuantum(gamma*MagickOver_((double) q[i],beta,
+ (double) p->red,alpha));
break;
}
case GreenPixelChannel:
{
- composite[i]=ClampToQuantum(gamma*MagickOver_((MagickRealType) q[i],
- beta,(MagickRealType) p->green,alpha));
+ composite[i]=ClampToQuantum(gamma*MagickOver_((double) q[i],beta,
+ (double) p->green,alpha));
break;
}
case BluePixelChannel:
{
- composite[i]=ClampToQuantum(gamma*MagickOver_((MagickRealType) q[i],
- beta,(MagickRealType) p->blue,alpha));
+ composite[i]=ClampToQuantum(gamma*MagickOver_((double) q[i],beta,
+ (double) p->blue,alpha));
break;
}
case BlackPixelChannel:
{
- composite[i]=ClampToQuantum(gamma*MagickOver_((MagickRealType) q[i],
- beta,(MagickRealType) p->black,alpha));
+ composite[i]=ClampToQuantum(gamma*MagickOver_((double) q[i],beta,
+ (double) p->black,alpha));
break;
}
case AlphaPixelChannel:
}
MagickExport MagickBooleanType SetImageAlphaChannel(Image *image,
- const AlphaChannelType alpha_type,ExceptionInfo *exception)
+ const AlphaChannelOption alpha_type,ExceptionInfo *exception)
{
MagickBooleanType
status;
{
case ActivateAlphaChannel:
{
- image->matte=MagickTrue;
+ image->alpha_trait=BlendPixelTrait;
break;
}
case BackgroundAlphaChannel:
/*
Set transparent pixels to background color.
*/
- if (image->matte == MagickFalse)
+ if (image->alpha_trait != BlendPixelTrait)
break;
if (SetImageStorageClass(image,DirectClass,exception) == MagickFalse)
break;
image_view=AcquireAuthenticCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
#pragma omp parallel for schedule(static,4) shared(status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
for (x=0; x < (ssize_t) image->columns; x++)
{
if (GetPixelAlpha(image,q) == TransparentAlpha)
- SetPixelInfoPixel(image,&image->background_color,q);
+ {
+ SetPixelInfoPixel(image,&image->background_color,q);
+ SetPixelChannel(image,AlphaPixelChannel,TransparentAlpha,q);
+ }
q+=GetPixelChannels(image);
}
if (SyncCacheViewAuthenticPixels(image_view,exception) == MagickFalse)
}
case DeactivateAlphaChannel:
{
- image->matte=MagickFalse;
+ image->alpha_trait=CopyPixelTrait;
break;
}
case ExtractAlphaChannel:
{
status=CompositeImage(image,image,AlphaCompositeOp,MagickTrue,0,0,
exception);
- image->matte=MagickFalse;
+ image->alpha_trait=CopyPixelTrait;
break;
}
case OpaqueAlphaChannel:
/*
Remove transparency.
*/
- if (image->matte == MagickFalse)
+ if (image->alpha_trait != BlendPixelTrait)
break;
if (SetImageStorageClass(image,DirectClass,exception) == MagickFalse)
break;
image_view=AcquireAuthenticCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
#pragma omp parallel for schedule(static,4) shared(status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
for (x=0; x < (ssize_t) image->columns; x++)
{
FlattenPixelInfo(image,&image->background_color,
- image->background_color.alpha,q,(MagickRealType)
+ image->background_color.alpha,q,(double)
GetPixelAlpha(image,q),q);
q+=GetPixelChannels(image);
}
status=MagickFalse;
}
image_view=DestroyCacheView(image_view);
- image->matte=image->background_color.matte;
+ image->alpha_trait=image->background_color.alpha_trait;
return(status);
}
case SetAlphaChannel:
{
- if (image->matte == MagickFalse)
+ if (image->alpha_trait != BlendPixelTrait)
status=SetImageAlpha(image,OpaqueAlpha,exception);
break;
}
assert(image->signature == MagickSignature);
if (SetImageStorageClass(image,DirectClass,exception) == MagickFalse)
return(MagickFalse);
- if ((IsGrayColorspace(image->colorspace) != MagickFalse) &&
- (IsPixelInfoGray(&image->background_color) == MagickFalse))
+ if ((IsPixelInfoGray(&image->background_color) == MagickFalse) &&
+ (IsGrayColorspace(image->colorspace) != MagickFalse))
(void) TransformImageColorspace(image,sRGBColorspace,exception);
- if ((image->background_color.matte != MagickFalse) &&
- (image->matte == MagickFalse))
+ if ((image->background_color.alpha_trait == BlendPixelTrait) &&
+ (image->alpha_trait != BlendPixelTrait))
(void) SetImageAlpha(image,OpaqueAlpha,exception);
/*
Set image background color.
% %
% %
% %
+% S e t I m a g e C h a n n e l M a s k %
+% %
+% %
+% %
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%
+% SetImageChannelMask() sets the image channel mask from the specified channel
+% mask.
+%
+% The format of the SetImageChannelMask method is:
+%
+% ChannelType SetImageChannelMask(Image *image,
+% const ChannelType channel_mask)
+%
+% A description of each parameter follows:
+%
+% o image: the image.
+%
+% o channel_mask: the channel mask.
+%
+*/
+MagickExport ChannelType SetImageChannelMask(Image *image,
+ const ChannelType channel_mask)
+{
+ ChannelType
+ mask;
+
+ mask=image->channel_mask;
+ image->channel_mask=channel_mask;
+ SetPixelChannelMask(image,channel_mask);
+ return(mask);
+}
+\f
+/*
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+% %
+% %
+% %
% S e t I m a g e C o l o r %
% %
% %
assert(image->signature == MagickSignature);
assert(color != (const PixelInfo *) NULL);
image->colorspace=color->colorspace;
- image->matte=color->matte;
+ image->alpha_trait=color->alpha_trait;
image->fuzz=color->fuzz;
image->depth=color->depth;
status=MagickTrue;
image_view=AcquireAuthenticCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
#pragma omp parallel for schedule(static,4) shared(status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
}
(void) ResetMagickMemory(magick,0,sizeof(magick));
count=ReadBlob(image,2*MaxTextExtent,magick);
+ (void) SeekBlob(image,-((MagickOffsetType) count),SEEK_CUR);
(void) CloseBlob(image);
image=DestroyImage(image);
/*
if (mask == (const Image *) NULL)
{
image->mask=MagickFalse;
- return(MagickTrue);
+ return(SyncImagePixelCache(image,exception));
}
- status=MagickTrue;
image->mask=MagickTrue;
+ if (SyncImagePixelCache(image,exception) == MagickFalse)
+ return(MagickFalse);
+ status=MagickTrue;
mask_view=AcquireVirtualCacheView(mask,exception);
image_view=AcquireAuthenticCacheView(image,exception);
+#if defined(MAGICKCORE_OPENMP_SUPPORT)
+ #pragma omp parallel for schedule(static,4) shared(status) \
+ magick_threads(mask,image,1,1)
+#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
register const Quantum
}
for (x=0; x < (ssize_t) image->columns; x++)
{
- SetPixelMask(image,GetPixelGray(mask,p),q);
+ SetPixelMask(image,ClampToQuantum(GetPixelIntensity(mask,p)),q);
p+=GetPixelChannels(mask);
q+=GetPixelChannels(image);
}
if (image->debug != MagickFalse)
(void) LogMagickEvent(TraceEvent,GetMagickModule(),"...");
assert(image->signature == MagickSignature);
- image->matte=MagickTrue;
+ image->alpha_trait=BlendPixelTrait;
status=MagickTrue;
image_view=AcquireAuthenticCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
#pragma omp parallel for schedule(static,4) shared(status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
% %
% %
% %
-% S e t I m a g e T y p e %
-% %
-% %
-% %
-%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
-%
-% SetImageType() sets the type of image. Choose from these types:
-%
-% Bilevel Grayscale GrayscaleMatte
-% Palette PaletteMatte TrueColor
-% TrueColorMatte ColorSeparation ColorSeparationMatte
-% OptimizeType
-%
-% The format of the SetImageType method is:
-%
-% MagickBooleanType SetImageType(Image *image,const ImageType type,
-% ExceptionInfo *exception)
-%
-% A description of each parameter follows:
-%
-% o image: the image.
-%
-% o type: Image type.
-%
-% o exception: return any errors or warnings in this structure.
-%
-*/
-MagickExport MagickBooleanType SetImageType(Image *image,const ImageType type,
- ExceptionInfo *exception)
-{
- const char
- *artifact;
-
- ImageInfo
- *image_info;
-
- MagickBooleanType
- status;
-
- QuantizeInfo
- *quantize_info;
-
- assert(image != (Image *) NULL);
- if (image->debug != MagickFalse)
- (void) LogMagickEvent(TraceEvent,GetMagickModule(),"...");
- assert(image->signature == MagickSignature);
- status=MagickTrue;
- image_info=AcquireImageInfo();
- image_info->dither=image->dither;
- artifact=GetImageArtifact(image,"dither");
- if (artifact != (const char *) NULL)
- (void) SetImageOption(image_info,"dither",artifact);
- switch (type)
- {
- case BilevelType:
- {
- if (IsImageMonochrome(image,exception) == MagickFalse)
- {
- quantize_info=AcquireQuantizeInfo(image_info);
- quantize_info->number_colors=2;
- quantize_info->colorspace=GRAYColorspace;
- status=QuantizeImage(quantize_info,image,exception);
- quantize_info=DestroyQuantizeInfo(quantize_info);
- }
- image->matte=MagickFalse;
- break;
- }
- case GrayscaleType:
- {
- if (IsImageGray(image,exception) == MagickFalse)
- status=TransformImageColorspace(image,GRAYColorspace,exception);
- image->matte=MagickFalse;
- break;
- }
- case GrayscaleMatteType:
- {
- if (IsImageGray(image,exception) == MagickFalse)
- status=TransformImageColorspace(image,GRAYColorspace,exception);
- if (image->matte == MagickFalse)
- (void) SetImageAlphaChannel(image,OpaqueAlphaChannel,exception);
- break;
- }
- case PaletteType:
- {
- if (IssRGBColorspace(image->colorspace) == MagickFalse)
- status=TransformImageColorspace(image,sRGBColorspace,exception);
- if ((image->storage_class == DirectClass) || (image->colors > 256))
- {
- quantize_info=AcquireQuantizeInfo(image_info);
- quantize_info->number_colors=256;
- status=QuantizeImage(quantize_info,image,exception);
- quantize_info=DestroyQuantizeInfo(quantize_info);
- }
- image->matte=MagickFalse;
- break;
- }
- case PaletteBilevelMatteType:
- {
- ChannelType
- channel_mask;
-
- if (IssRGBColorspace(image->colorspace) == MagickFalse)
- status=TransformImageColorspace(image,sRGBColorspace,exception);
- if (image->matte == MagickFalse)
- (void) SetImageAlphaChannel(image,OpaqueAlphaChannel,exception);
- channel_mask=SetPixelChannelMask(image,AlphaChannel);
- (void) BilevelImage(image,(double) QuantumRange/2.0,exception);
- (void) SetPixelChannelMask(image,channel_mask);
- quantize_info=AcquireQuantizeInfo(image_info);
- status=QuantizeImage(quantize_info,image,exception);
- quantize_info=DestroyQuantizeInfo(quantize_info);
- break;
- }
- case PaletteMatteType:
- {
- if (IssRGBColorspace(image->colorspace) == MagickFalse)
- status=TransformImageColorspace(image,sRGBColorspace,exception);
- if (image->matte == MagickFalse)
- (void) SetImageAlphaChannel(image,OpaqueAlphaChannel,exception);
- quantize_info=AcquireQuantizeInfo(image_info);
- quantize_info->colorspace=TransparentColorspace;
- status=QuantizeImage(quantize_info,image,exception);
- quantize_info=DestroyQuantizeInfo(quantize_info);
- break;
- }
- case TrueColorType:
- {
- if (IssRGBColorspace(image->colorspace) == MagickFalse)
- status=TransformImageColorspace(image,sRGBColorspace,exception);
- if (image->storage_class != DirectClass)
- status=SetImageStorageClass(image,DirectClass,exception);
- image->matte=MagickFalse;
- break;
- }
- case TrueColorMatteType:
- {
- if (IssRGBColorspace(image->colorspace) == MagickFalse)
- status=TransformImageColorspace(image,sRGBColorspace,exception);
- if (image->storage_class != DirectClass)
- status=SetImageStorageClass(image,DirectClass,exception);
- if (image->matte == MagickFalse)
- (void) SetImageAlphaChannel(image,OpaqueAlphaChannel,exception);
- break;
- }
- case ColorSeparationType:
- {
- if (image->colorspace != CMYKColorspace)
- {
- if (IssRGBColorspace(image->colorspace) == MagickFalse)
- status=TransformImageColorspace(image,sRGBColorspace,exception);
- status=TransformImageColorspace(image,CMYKColorspace,exception);
- }
- if (image->storage_class != DirectClass)
- status=SetImageStorageClass(image,DirectClass,exception);
- image->matte=MagickFalse;
- break;
- }
- case ColorSeparationMatteType:
- {
- if (image->colorspace != CMYKColorspace)
- {
- if (IssRGBColorspace(image->colorspace) == MagickFalse)
- status=TransformImageColorspace(image,sRGBColorspace,exception);
- status=TransformImageColorspace(image,CMYKColorspace,exception);
- }
- if (image->storage_class != DirectClass)
- status=SetImageStorageClass(image,DirectClass,exception);
- if (image->matte == MagickFalse)
- status=SetImageAlphaChannel(image,OpaqueAlphaChannel,exception);
- break;
- }
- case OptimizeType:
- case UndefinedType:
- break;
- }
- image->type=type;
- image_info=DestroyImageInfo(image_info);
- return(status);
-}
-\f
-/*
-%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
-% %
-% %
-% %
% S e t I m a g e V i r t u a l P i x e l M e t h o d %
% %
% %
*smush_image;
MagickBooleanType
- matte,
proceed,
status;
MagickOffsetType
n;
+ PixelTrait
+ alpha_trait;
+
RectangleInfo
geometry;
assert(exception != (ExceptionInfo *) NULL);
assert(exception->signature == MagickSignature);
image=images;
- matte=image->matte;
+ alpha_trait=image->alpha_trait;
number_images=1;
width=image->columns;
height=image->rows;
next=GetNextImageInList(image);
for ( ; next != (Image *) NULL; next=GetNextImageInList(next))
{
- if (next->matte != MagickFalse)
- matte=MagickTrue;
+ if (next->alpha_trait == BlendPixelTrait)
+ alpha_trait=BlendPixelTrait;
number_images++;
if (stack != MagickFalse)
{
smush_image=DestroyImage(smush_image);
return((Image *) NULL);
}
- smush_image->matte=matte;
+ smush_image->alpha_trait=alpha_trait;
(void) SetImageBackgroundColor(smush_image,exception);
status=MagickTrue;
x_offset=0;
if ((size_t) index < image->colors)
return(index);
*range_exception=MagickTrue;
- return(0);
+ return((Quantum) 0);
}
MagickExport MagickBooleanType SyncImage(Image *image,ExceptionInfo *exception)
image_view=AcquireAuthenticCacheView(image,exception);
#if defined(MAGICKCORE_OPENMP_SUPPORT)
#pragma omp parallel for schedule(static,4) shared(range_exception,status) \
- dynamic_number_threads(image->columns,image->rows,1)
+ magick_threads(image,image,image->rows,1)
#endif
for (y=0; y < (ssize_t) image->rows; y++)
{
status=MagickFalse;
}
image_view=DestroyCacheView(image_view);
- if (range_exception != MagickFalse)
+ if ((image->ping == MagickFalse) && (range_exception != MagickFalse))
(void) ThrowMagickException(exception,GetMagickModule(),CorruptImageError,
- "InvalidColormapIndex","'%s'",image->filename);
+ "InvalidColormapIndex","`%s'",image->filename);
return(status);
}
\f
% %
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%
-% SyncImageSettings() syncs the image_info options into per-image attributes.
+% SyncImageSettings() syncs any image_info global options into per-image
+% attributes.
+%
+% Note: in IMv6 free form 'options' were always mapped into 'artifacts', so
+% that operations and coders can find such settings. In IMv7 if a desired
+% per-image artifact is not set, then it will directly look for a global
+% option as a fallback, as such this copy is no longer needed, only the
+% link set up.
%
% The format of the SyncImageSettings method is:
%
MagickExport MagickBooleanType SyncImageSettings(const ImageInfo *image_info,
Image *image,ExceptionInfo *exception)
{
- char
- property[MaxTextExtent];
-
const char
- *option,
- *value;
+ *option;
GeometryInfo
geometry_info;
exception);
option=GetImageOption(image_info,"channel");
if (option != (const char *) NULL)
- (void) SetPixelChannelMapMask(image,(ChannelType)
+ (void) SetPixelChannelMask(image,(ChannelType)
ParseChannelOption(option));
/* FUTURE: do not sync compose to per-image compose setting here */
option=GetImageOption(image_info,"compose");
image->chromaticity.white_point.y=image->chromaticity.white_point.x;
}
ResetImageOptionIterator(image_info);
- for (option=GetNextImageOption(image_info); option != (const char *) NULL; )
+#if 0
{
- value=GetImageOption(image_info,option);
- if (value != (const char *) NULL)
- {
- (void) FormatLocaleString(property,MaxTextExtent,"%s",option);
- (void) SetImageArtifact(image,property,value);
- }
- option=GetNextImageOption(image_info);
+ /* IMv6: Copy freeform global options into per-image artifacts, so
+ * various operations and coders can access them.
+ *
+ * This has a problem, as per-image artefacts may have been set in
+ * parenthesis, but may not be unset when parenthesis ends.
+ */
+ char
+ property[MaxTextExtent];
+
+ const char
+ *value;
+
+ for (option=GetNextImageOption(image_info); option != (const char *) NULL; )
+ {
+ value=GetImageOption(image_info,option);
+ if (value != (const char *) NULL)
+ {
+ (void) FormatLocaleString(property,MaxTextExtent,"%s",option);
+ (void) SetImageArtifact(image,property,value);
+ }
+ option=GetNextImageOption(image_info);
+ }
}
+#else
+ /* IMv7: pointer to allow the lookup of pre-image artefact will fallback to
+ a global option setting/define. This saves a lot of duplication of
+ global options into per-image artifacts, while ensuring only specifically
+ set per-image artifacts are preverved when parenthesis ends.
+
+ This pointer is never explictally freed, as it is only used as a back
+ reference, not as the main pointer to the image_info structure. Images
+ being removed from a image_info image list (or yet to be added to such),
+ should have this pointer reset to NULL.
+ */
+ image->image_info=image_info;
+#endif
return(MagickTrue);
}