commit 19eb64195ef744f61293025952df1e5e6de66524
author cristy <urban-warrior@git.imagemagick.org> Fri Apr 23 14:42:29 2010 +0000
committer cristy <urban-warrior@git.imagemagick.org> Fri Apr 23 14:42:29 2010 +0000
tree cf488190542752eea6120643ebf28b596c67c729
parent aceefe6cc7efae5c35441381643b42a52b865731

magick/histogram.c

diff --git a/magick/histogram.c b/magick/histogram.c
index 4de6017..dd42ad3 100644
--- a/magick/histogram.c
+++ b/magick/histogram.c
@@ -987,7 +987,8 @@
   const ChannelType channel,const double black_value,const double white_value)
 {
   double
-    min,max;
+    min,
+    max;
 
   MagickStatusType
     status;
@@ -1001,9 +1002,9 @@
       (void) GetImageChannelRange(image,channel,&min,&max,&image->exception);
       min+=black_value;
       max-=white_value;
-      if ( fabs(min-max) >= MagickEpsilon )
-        status = LevelImageChannel(image,channel,min,max,1.0);
-      return(status);
+      if (fabs(min-max) >= MagickEpsilon)
+        status&=LevelImageChannel(image,channel,min,max,1.0);
+      return(status != 0 ? MagickTrue : MagickFalse);
     }
   /*
     Auto-level each channel separately.
@@ -1013,7 +1014,7 @@
       (void) GetImageChannelRange(image,RedChannel,&min,&max,&image->exception);
       min+=black_value;
       max-=white_value;
-      if ( fabs(min-max) >= MagickEpsilon )
+      if (fabs(min-max) >= MagickEpsilon)
         status&=LevelImageChannel(image,RedChannel,min,max,1.0);
     }
   if ((channel & GreenChannel) != 0)
@@ -1022,7 +1023,7 @@
         &image->exception);
       min+=black_value;
       max-=white_value;
-      if ( fabs(min-max) >= MagickEpsilon )
+      if (fabs(min-max) >= MagickEpsilon)
         status&=LevelImageChannel(image,GreenChannel,min,max,1.0);
     }
   if ((channel & BlueChannel) != 0)
@@ -1031,7 +1032,7 @@
         &image->exception);
       min+=black_value;
       max-=white_value;
-      if ( fabs(min-max) >= MagickEpsilon )
+      if (fabs(min-max) >= MagickEpsilon)
         status&=LevelImageChannel(image,BlueChannel,min,max,1.0);
     }
   if (((channel & OpacityChannel) != 0) &&
@@ -1041,7 +1042,7 @@
         &image->exception);
       min+=black_value;
       max-=white_value;
-      if ( fabs(min-max) >= MagickEpsilon )
+      if (fabs(min-max) >= MagickEpsilon)
         status&=LevelImageChannel(image,OpacityChannel,min,max,1.0);
     }
   if (((channel & IndexChannel) != 0) &&
@@ -1051,10 +1052,10 @@
         &image->exception);
       min+=black_value;
       max-=white_value;
-      if ( fabs(min-max) >= MagickEpsilon )
+      if (fabs(min-max) >= MagickEpsilon)
         status&=LevelImageChannel(image,IndexChannel,min,max,1.0);
     }
-  return(status);
+  return(status != 0 ? MagickTrue : MagickFalse);
 }
 
 /*