#include "magick/memory_.h"
#include "magick/monitor.h"
#include "magick/monitor-private.h"
+#include "magick/option.h"
#include "magick/resource_.h"
#include "magick/quantum-private.h"
#include "magick/static.h"
#include "magick/statistic.h"
#include "magick/string_.h"
#include "magick/module.h"
+#include "magick/utility.h"
\f
/*
Forward declarations.
%
% The format of the RegisterHISTOGRAMImage method is:
%
-% unsigned long RegisterHISTOGRAMImage(void)
+% size_t RegisterHISTOGRAMImage(void)
%
*/
-ModuleExport unsigned long RegisterHISTOGRAMImage(void)
+ModuleExport size_t RegisterHISTOGRAMImage(void)
{
MagickInfo
*entry;
entry=SetMagickInfo("HISTOGRAM");
entry->encoder=(EncodeImageHandler *) WriteHISTOGRAMImage;
entry->adjoin=MagickFalse;
- entry->format_type=ExplicitFormatType;
+ entry->format_type=ImplicitFormatType;
entry->description=ConstantString("Histogram of the image");
entry->module=ConstantString("HISTOGRAM");
(void) RegisterMagickInfo(entry);
static MagickBooleanType WriteHISTOGRAMImage(const ImageInfo *image_info,
Image *image)
{
-#define HistogramDensity "256x200"
+#define HistogramDensity "256x100"
ChannelType
channel;
char
filename[MaxTextExtent];
+ const char
+ *option;
+
ExceptionInfo
*exception;
- FILE
- *file;
-
Image
*histogram_image;
ImageInfo
*write_info;
- int
- unique_file;
-
- long
+ ssize_t
y;
MagickBooleanType
register const PixelPacket
*p;
- register long
+ register ssize_t
x;
register PixelPacket
*/
channel=image_info->channel;
(void) ResetMagickMemory(histogram,0,length*sizeof(*histogram));
- for (y=0; y < (long) image->rows; y++)
+ for (y=0; y < (ssize_t) image->rows; y++)
{
p=GetVirtualPixels(image,0,y,image->columns,1,&image->exception);
if (p == (const PixelPacket *) NULL)
break;
- for (x=0; x < (long) image->columns; x++)
+ for (x=0; x < (ssize_t) image->columns; x++)
{
if ((channel & RedChannel) != 0)
histogram[ScaleQuantumToChar(GetRedPixelComponent(p))].red++;
}
}
maximum=histogram[0].red;
- for (x=0; x < (long) histogram_image->columns; x++)
+ for (x=0; x < (ssize_t) histogram_image->columns; x++)
{
if (((channel & RedChannel) != 0) && (maximum < histogram[x].red))
maximum=histogram[x].red;
(void) QueryColorDatabase("#000000",&histogram_image->background_color,
&image->exception);
(void) SetImageBackgroundColor(histogram_image);
- for (x=0; x < (long) histogram_image->columns; x++)
+ for (x=0; x < (ssize_t) histogram_image->columns; x++)
{
q=GetAuthenticPixels(histogram_image,x,0,1,histogram_image->rows,exception);
if (q == (PixelPacket *) NULL)
break;
if ((channel & RedChannel) != 0)
{
- y=(long) (histogram_image->rows-scale*histogram[x].red+0.5);
+ y=(ssize_t) ceil(histogram_image->rows-scale*histogram[x].red-0.5);
r=q+y;
- for ( ; y < (long) histogram_image->rows; y++)
+ for ( ; y < (ssize_t) histogram_image->rows; y++)
{
r->red=(Quantum) QuantumRange;
r++;
}
if ((channel & GreenChannel) != 0)
{
- y=(long) (histogram_image->rows-scale*histogram[x].green+0.5);
+ y=(ssize_t) ceil(histogram_image->rows-scale*histogram[x].green-0.5);
r=q+y;
- for ( ; y < (long) histogram_image->rows; y++)
+ for ( ; y < (ssize_t) histogram_image->rows; y++)
{
r->green=(Quantum) QuantumRange;
r++;
}
if ((channel & BlueChannel) != 0)
{
- y=(long) (histogram_image->rows-scale*histogram[x].blue+0.5);
+ y=(ssize_t) ceil(histogram_image->rows-scale*histogram[x].blue-0.5);
r=q+y;
- for ( ; y < (long) histogram_image->rows; y++)
+ for ( ; y < (ssize_t) histogram_image->rows; y++)
{
r->blue=(Quantum) QuantumRange;
r++;
Relinquish resources.
*/
histogram=(MagickPixelPacket *) RelinquishMagickMemory(histogram);
- file=(FILE *) NULL;
- unique_file=AcquireUniqueFileResource(filename);
- if (unique_file != -1)
- file=fdopen(unique_file,"wb");
- if ((unique_file != -1) && (file != (FILE *) NULL))
+ option=GetImageOption(image_info,"histogram:unique-colors");
+ if ((option == (const char *) NULL) || (IsMagickTrue(option) != MagickFalse))
{
- char
- *property;
+ FILE
+ *file;
+
+ int
+ unique_file;
/*
- Add a histogram as an image comment.
+ Add a unique colors as an image comment.
*/
- (void) GetNumberColors(image,file,&image->exception);
- (void) fclose(file);
- property=FileToString(filename,~0UL,&image->exception);
- if (property != (char *) NULL)
+ file=(FILE *) NULL;
+ unique_file=AcquireUniqueFileResource(filename);
+ if (unique_file != -1)
+ file=fdopen(unique_file,"wb");
+ if ((unique_file != -1) && (file != (FILE *) NULL))
{
- (void) SetImageProperty(histogram_image,"comment",property);
- property=DestroyString(property);
+ char
+ *property;
+
+ (void) GetNumberColors(image,file,&image->exception);
+ (void) fclose(file);
+ property=FileToString(filename,~0UL,&image->exception);
+ if (property != (char *) NULL)
+ {
+ (void) SetImageProperty(histogram_image,"comment",property);
+ property=DestroyString(property);
+ }
}
+ (void) RelinquishUniqueFileResource(filename);
}
- (void) RelinquishUniqueFileResource(filename);
/*
Write Histogram image.
*/
(void) CopyMagickString(histogram_image->filename,image_info->filename,
MaxTextExtent);
write_info=CloneImageInfo(image_info);
- (void) SetImageInfo(write_info,MagickTrue,&image->exception);
+ (void) SetImageInfo(write_info,1,&image->exception);
if (LocaleCompare(write_info->magick,"HISTOGRAM") == 0)
(void) FormatMagickString(histogram_image->filename,MaxTextExtent,
"miff:%s",write_info->filename);