]> granicus.if.org Git - imagemagick/commitdiff
(no commit message)
authorcristy <urban-warrior@git.imagemagick.org>
Thu, 18 Apr 2013 11:47:41 +0000 (11:47 +0000)
committercristy <urban-warrior@git.imagemagick.org>
Thu, 18 Apr 2013 11:47:41 +0000 (11:47 +0000)
MagickCore/display.c
MagickCore/random.c
MagickCore/resource.c
MagickCore/utility-private.h
MagickCore/utility.c

index b01bb6cce005bb78a010ace822ff262adb7c6ee5..e437308ad232718275067414f2cc5ccb3d055acb 100644 (file)
@@ -1675,7 +1675,7 @@ MagickExport MagickBooleanType DisplayImages(const ImageInfo *image_info,
   assert(image_info->signature == MagickSignature);
   assert(images != (Image *) NULL);
   assert(images->signature == MagickSignature);
-  ifIfMagickTrue(images->debug) )
+  if (IfMagickTrue(images->debug) )
     (void) LogMagickEvent(TraceEvent,GetMagickModule(),"%s",images->filename);
   display=XOpenDisplay(image_info->server_name);
   if (display == (Display *) NULL)
@@ -1934,7 +1934,7 @@ static MagickBooleanType XAnnotateEditImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -2027,7 +2027,7 @@ static MagickBooleanType XAnnotateEditImage(Display *display,
               break;
             transparent_pen=pen_number == (MaxNumberPens-2) ? MagickTrue :
               MagickFalse;
-            ifIfMagickTrue(transparent_pen) )
+            if (IfMagickTrue(transparent_pen) )
               break;
             if (pen_number == (MaxNumberPens-1))
               {
@@ -2072,7 +2072,7 @@ static MagickBooleanType XAnnotateEditImage(Display *display,
               break;
             transparent_box=pen_number == (MaxNumberPens-2) ? MagickTrue :
               MagickFalse;
-            ifIfMagickTrue(transparent_box) )
+            if (IfMagickTrue(transparent_box) )
               break;
             if (pen_number == (MaxNumberPens-1))
               {
@@ -2223,7 +2223,7 @@ static MagickBooleanType XAnnotateEditImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -2271,10 +2271,10 @@ static MagickBooleanType XAnnotateEditImage(Display *display,
   XGetAnnotateInfo(annotate_info);
   annotate_info->x=x;
   annotate_info->y=y;
-  ifIfMagickFalse(transparent_box) && IfMagickFalse(transparent_pen))
+  if (IfMagickFalse(transparent_box) && IfMagickFalse(transparent_pen))
     annotate_info->stencil=OpaqueStencil;
   else
-    ifIfMagickFalse(transparent_box) )
+    if (IfMagickFalse(transparent_box) )
       annotate_info->stencil=BackgroundStencil;
     else
       annotate_info->stencil=ForegroundStencil;
@@ -2945,7 +2945,7 @@ static MagickBooleanType XChopImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -3084,7 +3084,7 @@ static MagickBooleanType XChopImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -3120,7 +3120,7 @@ static MagickBooleanType XChopImage(Display *display,
         /*
           Display info and draw chopping line.
         */
-        ifIfMagickFalse(windows->info.mapped) )
+        if (IfMagickFalse(windows->info.mapped) )
           (void) XMapWindow(display,windows->info.id);
         (void) FormatLocaleString(text,MaxTextExtent,
           " %.20gx%.20g%+.20g%+.20g",(double) chop_info.width,(double)
@@ -3130,7 +3130,7 @@ static MagickBooleanType XChopImage(Display *display,
           windows->image.highlight_context,&segment_info);
       }
     else
-      ifIfMagickTrue(windows->info.mapped) )
+      if (IfMagickTrue(windows->info.mapped) )
         (void) XWithdrawWindow(display,windows->info.id,windows->info.screen);
     /*
       Wait for next event.
@@ -3387,7 +3387,7 @@ static MagickBooleanType XColorEditImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -3679,7 +3679,7 @@ static MagickBooleanType XColorEditImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -3743,7 +3743,7 @@ static MagickBooleanType XColorEditImage(Display *display,
             /*
               Update color information using point algorithm.
             */
-            ifIfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
+            if (IfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
               return(MagickFalse);
             q=GetCacheViewAuthenticPixels(image_view,(ssize_t)x_offset,
               (ssize_t) y_offset,1,1,exception);
@@ -3788,7 +3788,7 @@ static MagickBooleanType XColorEditImage(Display *display,
                       }
                     q+=GetPixelChannels(*image);
                   }
-                  ifIfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
+                  if (IfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
                     break;
                 }
               }
@@ -3847,7 +3847,7 @@ static MagickBooleanType XColorEditImage(Display *display,
             /*
               Update color information using reset algorithm.
             */
-            ifIfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
+            if (IfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
               return(MagickFalse);
             for (y=0; y < (int) (*image)->rows; y++)
             {
@@ -3862,7 +3862,7 @@ static MagickBooleanType XColorEditImage(Display *display,
                 SetPixelBlue(*image,ScaleShortToQuantum(color.blue),q);
                 q+=GetPixelChannels(*image);
               }
-              ifIfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
+              if (IfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
                 break;
             }
             break;
@@ -4022,7 +4022,7 @@ static MagickBooleanType XCompositeImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -4136,7 +4136,7 @@ static MagickBooleanType XCompositeImage(Display *display,
     {
       case ButtonPress:
       {
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Button Press: 0x%lx %u +%d+%d",event.xbutton.window,
             event.xbutton.button,event.xbutton.x,event.xbutton.y);
@@ -4156,7 +4156,7 @@ static MagickBooleanType XCompositeImage(Display *display,
       }
       case ButtonRelease:
       {
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Button Release: 0x%lx %u +%d+%d",event.xbutton.window,
             event.xbutton.button,event.xbutton.x,event.xbutton.y);
@@ -4196,7 +4196,7 @@ static MagickBooleanType XCompositeImage(Display *display,
         length=XLookupString((XKeyEvent *) &event.xkey,command,(int)
           sizeof(command),&key_symbol,(XComposeStatus *) NULL);
         *(command+length)='\0';
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Key press: 0x%lx (%s)",(unsigned long) key_symbol,command);
         switch ((int) key_symbol)
@@ -4238,7 +4238,7 @@ static MagickBooleanType XCompositeImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -4255,7 +4255,7 @@ static MagickBooleanType XCompositeImage(Display *display,
       }
       default:
       {
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),"Event type: %d",
             event.type);
         break;
@@ -4333,7 +4333,7 @@ static MagickBooleanType XCompositeImage(Display *display,
       (void) SetImageAlphaChannel(composite_image,OpaqueAlphaChannel,exception);
       opacity=(Quantum) (ScaleQuantumToChar(QuantumRange)-
         ((ssize_t) ScaleQuantumToChar(QuantumRange)*blend)/100);
-      ifIfMagickFalse(SetImageStorageClass(image,DirectClass,exception)) )
+      if (IfMagickFalse(SetImageStorageClass(image,DirectClass,exception)) )
         return(MagickFalse);
       image->alpha_trait=BlendPixelTrait;
       image_view=AcquireAuthenticCacheView(image,exception);
@@ -4348,7 +4348,7 @@ static MagickBooleanType XCompositeImage(Display *display,
           SetPixelAlpha(image,opacity,q);
           q+=GetPixelChannels(image);
         }
-        ifIfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
+        if (IfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
           break;
       }
       image_view=DestroyCacheView(image_view);
@@ -4434,7 +4434,7 @@ static MagickBooleanType XConfigureImage(Display *display,
   */
   width=(unsigned int) windows->image.window_changes.width;
   height=(unsigned int) windows->image.window_changes.height;
-  ifIfMagickTrue(image->debug) )
+  if (IfMagickTrue(image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),
       "Configure Image: %dx%d=>%.20gx%.20g",windows->image.ximage->width,
       windows->image.ximage->height,(double) width,(double) height);
@@ -4459,7 +4459,7 @@ static MagickBooleanType XConfigureImage(Display *display,
     (height*windows->image.y/windows->image.ximage->height);
   status=XMakeImage(display,resource_info,&windows->image,image,
     (unsigned int) width,(unsigned int) height,exception);
-  ifIfMagickFalse(status) )
+  if (IfMagickFalse(status) )
     XNoticeWidget(display,windows,"Unable to configure X image:",
       windows->image.name);
   /*
@@ -4495,7 +4495,7 @@ static MagickBooleanType XConfigureImage(Display *display,
   /*
     Update Magnify window configuration.
   */
-  ifIfMagickTrue(windows->magnify.mapped) )
+  if (IfMagickTrue(windows->magnify.mapped) )
     XMakeMagnifyImage(display,windows,exception);
   windows->pan.crop_geometry=windows->image.crop_geometry;
   XBestIconSize(display,&windows->pan,image);
@@ -4693,7 +4693,7 @@ static MagickBooleanType XCropImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -4840,7 +4840,7 @@ static MagickBooleanType XCropImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -4891,7 +4891,7 @@ static MagickBooleanType XCropImage(Display *display,
           /*
             Display info and draw cropping rectangle.
           */
-          ifIfMagickFalse(windows->info.mapped) )
+          if (IfMagickFalse(windows->info.mapped) )
             (void) XMapWindow(display,windows->info.id);
           (void) FormatLocaleString(text,MaxTextExtent,
             " %.20gx%.20g%+.20g%+.20g",(double) crop_info.width,(double)
@@ -4901,7 +4901,7 @@ static MagickBooleanType XCropImage(Display *display,
             windows->image.highlight_context,&highlight_info);
         }
       else
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           (void) XWithdrawWindow(display,windows->info.id,windows->info.screen);
       /*
         Wait for next event.
@@ -4981,7 +4981,7 @@ static MagickBooleanType XCropImage(Display *display,
     (void) XMapWindow(display,windows->info.id);
     do
     {
-      ifIfMagickTrue(windows->info.mapped) )
+      if (IfMagickTrue(windows->info.mapped) )
         {
           /*
             Display pointer position.
@@ -5246,7 +5246,7 @@ static MagickBooleanType XCropImage(Display *display,
           */
           x=event.xmotion.x;
           y=event.xmotion.y;
-          ifIfMagickTrue(windows->info.mapped) )
+          if (IfMagickTrue(windows->info.mapped) )
             {
               if ((x < (int) (windows->info.x+windows->info.width)) &&
                   (y < (int) (windows->info.y+windows->info.height)))
@@ -5354,7 +5354,7 @@ static MagickBooleanType XCropImage(Display *display,
   /*
     Cut image.
   */
-  ifIfMagickFalse(SetImageStorageClass(image,DirectClass,exception)) )
+  if (IfMagickFalse(SetImageStorageClass(image,DirectClass,exception)) )
     return(MagickFalse);
   image->alpha_trait=BlendPixelTrait;
   image_view=AcquireAuthenticCacheView(image,exception);
@@ -5369,7 +5369,7 @@ static MagickBooleanType XCropImage(Display *display,
       SetPixelAlpha(image,TransparentAlpha,q);
       q+=GetPixelChannels(image);
     }
-    ifIfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
+    if (IfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
       break;
   }
   image_view=DestroyCacheView(image_view);
@@ -5541,7 +5541,7 @@ static MagickBooleanType XDrawEditImage(Display *display,
     state=DefaultState;
     do
     {
-      ifIfMagickTrue(windows->info.mapped) )
+      if (IfMagickTrue(windows->info.mapped) )
         {
           /*
             Display pointer position.
@@ -5620,7 +5620,7 @@ static MagickBooleanType XDrawEditImage(Display *display,
                 break;
               transparent=pen_number == (MaxNumberPens-2) ? MagickTrue :
                 MagickFalse;
-              ifIfMagickTrue(transparent) )
+              if (IfMagickTrue(transparent) )
                 {
                   draw_info.stencil=TransparentStencil;
                   break;
@@ -5901,7 +5901,7 @@ static MagickBooleanType XDrawEditImage(Display *display,
           */
           x=event.xmotion.x;
           y=event.xmotion.y;
-          ifIfMagickTrue(windows->info.mapped) )
+          if (IfMagickTrue(windows->info.mapped) )
             {
               if ((x < (int) (windows->info.x+windows->info.width)) &&
                   (y < (int) (windows->info.y+windows->info.height)))
@@ -5974,7 +5974,7 @@ static MagickBooleanType XDrawEditImage(Display *display,
                 windows->image.highlight_context,&line_info);
             }
           else
-            ifIfMagickTrue(windows->info.mapped) )
+            if (IfMagickTrue(windows->info.mapped) )
               (void) XWithdrawWindow(display,windows->info.id,
                 windows->info.screen);
           break;
@@ -5996,7 +5996,7 @@ static MagickBooleanType XDrawEditImage(Display *display,
                 windows->image.highlight_context,&rectangle_info);
             }
           else
-            ifIfMagickTrue(windows->info.mapped) )
+            if (IfMagickTrue(windows->info.mapped) )
               (void) XWithdrawWindow(display,windows->info.id,
                 windows->info.screen);
           break;
@@ -6020,7 +6020,7 @@ static MagickBooleanType XDrawEditImage(Display *display,
                 windows->image.highlight_context,&rectangle_info);
             }
           else
-            ifIfMagickTrue(windows->info.mapped) )
+            if (IfMagickTrue(windows->info.mapped) )
               (void) XWithdrawWindow(display,windows->info.id,
                 windows->info.screen);
           break;
@@ -6046,7 +6046,7 @@ static MagickBooleanType XDrawEditImage(Display *display,
                 windows->image.highlight_context,&line_info);
             }
           else
-            ifIfMagickTrue(windows->info.mapped) )
+            if (IfMagickTrue(windows->info.mapped) )
               (void) XWithdrawWindow(display,windows->info.id,
                 windows->info.screen);
           break;
@@ -6493,7 +6493,7 @@ static void XImageCache(Display *display,XResourceInfo *resource_info,
       redo_image=(*image);
       *image=cache_image->list;
       cache_image=DestroyImage(cache_image);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         return;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -6659,7 +6659,7 @@ static void XImageCache(Display *display,XResourceInfo *resource_info,
       *image=DestroyImage(*image);
       *image=redo_image;
       redo_image=NewImageList();
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         return;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7217,7 +7217,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Save image.
       */
       status=XSaveImage(display,resource_info,windows,*image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           char
             message[MaxTextExtent];
@@ -7237,7 +7237,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Print image.
       */
       status=XPrintImage(display,resource_info,windows,*image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           char
             message[MaxTextExtent];
@@ -7262,8 +7262,8 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XFileBrowserWidget(display,windows,"Delete",filename);
       if (*filename == '\0')
         break;
-      status=IsMagickTrue(remove_utf8(filename));
-      ifIfMagickTrue(status) )
+      status=ShredFile(filename);
+      if (IfMagickTrue(status) )
         XNoticeWidget(display,windows,"Unable to delete image file:",filename);
       break;
     }
@@ -7316,7 +7316,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       /*
         exit program.
       */
-      ifIfMagickFalse(resource_info->confirm_exit) )
+      if (IfMagickFalse(resource_info->confirm_exit) )
         XClientMessage(display,windows->image.id,windows->im_protocols,
           windows->im_exit,CurrentTime);
       else
@@ -7358,7 +7358,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Paste image.
       */
       status=XPasteImage(display,resource_info,windows,*image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to paste X image",
             (*image)->filename);
@@ -7506,7 +7506,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Chop image.
       */
       status=XChopImage(display,resource_info,windows,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to cut X image",
             (*image)->filename);
@@ -7544,7 +7544,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
           (void) FormatLocaleString(windows->image.crop_geometry,MaxTextExtent,
             "%ux%u%+d%+d",width,height,(int) (*image)->columns-(int) width-x,y);
         }
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
       break;
@@ -7579,7 +7579,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
           (void) FormatLocaleString(windows->image.crop_geometry,MaxTextExtent,
             "%ux%u%+d%+d",width,height,x,(int) (*image)->rows-(int) height-y);
         }
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
       break;
@@ -7590,7 +7590,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Rotate image 90 degrees clockwise.
       */
       status=XRotateImage(display,resource_info,windows,90.0,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to rotate X image",
             (*image)->filename);
@@ -7604,7 +7604,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Rotate image 90 degrees counter-clockwise.
       */
       status=XRotateImage(display,resource_info,windows,-90.0,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to rotate X image",
             (*image)->filename);
@@ -7618,7 +7618,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Rotate image.
       */
       status=XRotateImage(display,resource_info,windows,0.0,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to rotate X image",
             (*image)->filename);
@@ -7665,7 +7665,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       windows->image.window_changes.width=(int) (*image)->columns;
       windows->image.window_changes.height=(int) (*image)->rows;
@@ -7706,7 +7706,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       windows->image.window_changes.width=(int) (*image)->columns;
       windows->image.window_changes.height=(int) (*image)->rows;
@@ -7732,7 +7732,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Trim image.
       */
       status=XTrimImage(display,resource_info,windows,*image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to trim X image",
             (*image)->filename);
@@ -7762,7 +7762,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         MaxTextExtent);
       (void) ModulateImage(*image,modulate_factors,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7790,7 +7790,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         MaxTextExtent);
       (void) ModulateImage(*image,modulate_factors,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7817,7 +7817,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         MaxTextExtent);
       (void) ModulateImage(*image,modulate_factors,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7842,7 +7842,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       (void) GammaImage(*image,atof(factor),exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7857,7 +7857,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       (void) ContrastImage(*image,MagickTrue,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7872,7 +7872,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       (void) ContrastImage(*image,MagickFalse,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7911,7 +7911,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       (void) ContrastStretchImage(*image,black_point,white_point,
         exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7948,7 +7948,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       (void) SigmoidalContrastImage(*image,MagickTrue,geometry_info.rho,
         geometry_info.sigma,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7963,7 +7963,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       (void) NormalizeImage(*image,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7978,7 +7978,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       (void) EqualizeImage(*image,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -7993,7 +7993,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       (void) NegateImage(*image,MagickFalse,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8009,7 +8009,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       (void) SetImageType(*image,(*image)->alpha_trait != BlendPixelTrait ?
         GrayscaleType : GrayscaleMatteType,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8043,7 +8043,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8074,7 +8074,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         NoDitherMethod;
       (void) QuantizeImage(&quantize_info,*image,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8098,7 +8098,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8136,7 +8136,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8172,7 +8172,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8212,7 +8212,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8248,7 +8248,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8284,7 +8284,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8313,7 +8313,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       threshold=StringToDoubleInterval(factor,(double) QuantumRange+1.0);
       (void) BilevelImage(*image,threshold,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8348,7 +8348,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8383,7 +8383,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8424,7 +8424,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8452,7 +8452,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         exception);
       (void) RaiseImage(*image,&page_geometry,MagickTrue,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8481,7 +8481,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       (void) SegmentImage(*image,sRGBColorspace,MagickFalse,geometry_info.rho,
         geometry_info.sigma,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8519,7 +8519,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8548,7 +8548,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       threshold=StringToDoubleInterval(factor,(double) QuantumRange+1.0);
       (void) SolarizeImage(*image,threshold,exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8584,7 +8584,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8620,7 +8620,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8663,7 +8663,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8701,7 +8701,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8737,7 +8737,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8777,7 +8777,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       XConfigureImageColormap(display,resource_info,windows,*image,exception);
       (void) XConfigureImage(display,resource_info,windows,*image,exception);
@@ -8789,7 +8789,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Annotate the image with text.
       */
       status=XAnnotateEditImage(display,resource_info,windows,*image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to annotate X image",
             (*image)->filename);
@@ -8803,7 +8803,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Draw image.
       */
       status=XDrawEditImage(display,resource_info,windows,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to draw on the X image",
             (*image)->filename);
@@ -8817,7 +8817,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Color edit.
       */
       status=XColorEditImage(display,resource_info,windows,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to pixel edit X image",
             (*image)->filename);
@@ -8831,7 +8831,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Matte edit.
       */
       status=XMatteEditImage(display,resource_info,windows,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to matte edit X image",
             (*image)->filename);
@@ -8846,7 +8846,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       */
       status=XCompositeImage(display,resource_info,windows,*image,
         exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         {
           XNoticeWidget(display,windows,"Unable to composite X image",
             (*image)->filename);
@@ -8892,7 +8892,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       windows->image.window_changes.width=(int) (*image)->columns;
       windows->image.window_changes.height=(int) (*image)->rows;
@@ -8948,7 +8948,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         }
       CatchException(exception);
       XSetCursorState(display,windows,MagickFalse);
-      ifIfMagickTrue(windows->image.orphan) )
+      if (IfMagickTrue(windows->image.orphan) )
         break;
       windows->image.window_changes.width=(int) (*image)->columns;
       windows->image.window_changes.height=(int) (*image)->rows;
@@ -8998,7 +8998,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       status=InvokeDelegate(image_info,*image,"edit",(char *) NULL,
         exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         XNoticeWidget(display,windows,"Unable to edit image comment",
           (char *) NULL);
       else
@@ -9028,7 +9028,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       (void) FormatLocaleString((*image)->filename,MaxTextExtent,"launch:%s",
         filename);
       status=WriteImage(image_info,*image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         XNoticeWidget(display,windows,"Unable to launch image editor",
           (char *) NULL);
       else
@@ -9057,7 +9057,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       /*
         Zoom image.
       */
-      ifIfMagickTrue(windows->magnify.mapped) )
+      if (IfMagickTrue(windows->magnify.mapped) )
         (void) XRaiseWindow(display,windows->magnify.id);
       else
         {
@@ -9116,7 +9116,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         filename);
       status=WriteImage(image_info,preview_image,exception);
       preview_image=DestroyImage(preview_image);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         XNoticeWidget(display,windows,"Unable to show image preview",
           (*image)->filename);
       XDelay(display,1500);
@@ -9149,7 +9149,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         "show:%s",filename);
       status=WriteImage(image_info,histogram_image,exception);
       histogram_image=DestroyImage(histogram_image);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         XNoticeWidget(display,windows,"Unable to show histogram",
           (*image)->filename);
       XDelay(display,1500);
@@ -9188,7 +9188,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         filename);
       status=WriteImage(image_info,matte_image,exception);
       matte_image=DestroyImage(matte_image);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         XNoticeWidget(display,windows,"Unable to show matte",
           (*image)->filename);
       XDelay(display,1500);
@@ -9201,7 +9201,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Background image.
       */
       status=XBackgroundImage(display,resource_info,windows,image,exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         break;
       nexus=CloneImage(*image,0,0,MagickTrue,exception);
       if (nexus != (Image *) NULL)
@@ -9232,7 +9232,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
         Set user preferences.
       */
       status=XPreferencesWidget(display,resource_info,windows);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         break;
       nexus=CloneImage(*image,0,0,MagickTrue,exception);
       if (nexus != (Image *) NULL)
@@ -9287,7 +9287,7 @@ static Image *XMagickCommand(Display *display,XResourceInfo *resource_info,
       XCheckRefreshWindows(display,windows);
       status=InvokeDelegate(image_info,*image,"browse",(char *) NULL,
         exception);
-      ifIfMagickFalse(status) )
+      if (IfMagickFalse(status) )
         XNoticeWidget(display,windows,"Unable to browse documentation",
           (char *) NULL);
       XDelay(display,1500);
@@ -9371,7 +9371,7 @@ static void XMagnifyImage(Display *display,XWindows *windows,XEvent *event,
     /*
       Map and unmap Info widget as text cursor crosses its boundaries.
     */
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         if ((x < (int) (windows->info.x+windows->info.width)) &&
             (y < (int) (windows->info.y+windows->info.height)))
@@ -9381,7 +9381,7 @@ static void XMagnifyImage(Display *display,XWindows *windows,XEvent *event,
       if ((x > (int) (windows->info.x+windows->info.width)) ||
           (y > (int) (windows->info.y+windows->info.height)))
         (void) XMapWindow(display,windows->info.id);
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -9611,7 +9611,7 @@ static void XMakePanImage(Display *display,XResourceInfo *resource_info,
   windows->pan.y=(int) windows->image.y;
   status=XMakeImage(display,resource_info,&windows->pan,image,
     windows->pan.width,windows->pan.height,exception);
-  ifIfMagickFalse(status) )
+  if (IfMagickFalse(status) )
     ThrowXWindowFatalException(ResourceLimitError,
      "MemoryAllocationFailed",image->filename);
   (void) XSetWindowBackgroundPixmap(display,windows->pan.id,
@@ -9749,7 +9749,7 @@ static MagickBooleanType XMatteEditImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -10037,7 +10037,7 @@ static MagickBooleanType XMatteEditImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -10092,7 +10092,7 @@ static MagickBooleanType XMatteEditImage(Display *display,
         if ((x_offset >= (int) (*image)->columns) ||
             (y_offset >= (int) (*image)->rows))
           continue;
-        ifIfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
+        if (IfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
           return(MagickFalse);
         if ((*image)->alpha_trait != BlendPixelTrait)
           (void) SetImageAlphaChannel(*image,OpaqueAlphaChannel,exception);
@@ -10137,7 +10137,7 @@ static MagickBooleanType XMatteEditImage(Display *display,
                   SetPixelAlpha(*image,(Quantum) StringToLong(matte),q);
                 q+=GetPixelChannels(*image);
               }
-              ifIfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
+              if (IfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
                 break;
             }
             break;
@@ -10186,7 +10186,7 @@ static MagickBooleanType XMatteEditImage(Display *display,
             /*
               Update matte information using reset algorithm.
             */
-            ifIfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
+            if (IfMagickFalse(SetImageStorageClass(*image,DirectClass,exception)) )
               return(MagickFalse);
             for (y=0; y < (int) (*image)->rows; y++)
             {
@@ -10199,7 +10199,7 @@ static MagickBooleanType XMatteEditImage(Display *display,
                 SetPixelAlpha(*image,(Quantum) StringToLong(matte),q);
                 q+=GetPixelChannels(*image);
               }
-              ifIfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
+              if (IfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
                 break;
             }
             if (StringToLong(matte) == (long) OpaqueAlpha)
@@ -10270,7 +10270,7 @@ static Image *XOpenImage(Display *display,XResourceInfo *resource_info,
   /*
     Request file name from user.
   */
-  ifIfMagickFalse(command) )
+  if (IfMagickFalse(command) )
     XFileBrowserWidget(display,windows,"Open",filename);
   else
     {
@@ -10693,7 +10693,7 @@ static MagickBooleanType XPasteImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -10767,7 +10767,7 @@ static MagickBooleanType XPasteImage(Display *display,
     {
       case ButtonPress:
       {
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Button Press: 0x%lx %u +%d+%d",event.xbutton.window,
             event.xbutton.button,event.xbutton.x,event.xbutton.y);
@@ -10796,7 +10796,7 @@ static MagickBooleanType XPasteImage(Display *display,
       }
       case ButtonRelease:
       {
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Button Release: 0x%lx %u +%d+%d",event.xbutton.window,
             event.xbutton.button,event.xbutton.x,event.xbutton.y);
@@ -10836,7 +10836,7 @@ static MagickBooleanType XPasteImage(Display *display,
         length=XLookupString((XKeyEvent *) &event.xkey,command,(int)
           sizeof(command),&key_symbol,(XComposeStatus *) NULL);
         *(command+length)='\0';
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Key press: 0x%lx (%s)",(long) key_symbol,command);
         switch ((int) key_symbol)
@@ -10878,7 +10878,7 @@ static MagickBooleanType XPasteImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -10895,7 +10895,7 @@ static MagickBooleanType XPasteImage(Display *display,
       }
       default:
       {
-        ifIfMagickTrue(image->debug) )
+        if (IfMagickTrue(image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),"Event type: %d",
             event.type);
         break;
@@ -11339,7 +11339,7 @@ static MagickBooleanType XROIImage(Display *display,
   state=DefaultState;
   do
   {
-    ifIfMagickTrue(windows->info.mapped) )
+    if (IfMagickTrue(windows->info.mapped) )
       {
         /*
           Display pointer position.
@@ -11449,7 +11449,7 @@ static MagickBooleanType XROIImage(Display *display,
         */
         x=event.xmotion.x;
         y=event.xmotion.y;
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           {
             if ((x < (int) (windows->info.x+windows->info.width)) &&
                 (y < (int) (windows->info.y+windows->info.height)))
@@ -11500,7 +11500,7 @@ static MagickBooleanType XROIImage(Display *display,
           /*
             Display info and draw region of interest rectangle.
           */
-          ifIfMagickFalse(windows->info.mapped) )
+          if (IfMagickFalse(windows->info.mapped) )
             (void) XMapWindow(display,windows->info.id);
           (void) FormatLocaleString(text,MaxTextExtent,
             " %.20gx%.20g%+.20g%+.20g",(double) roi_info.width,(double)
@@ -11510,7 +11510,7 @@ static MagickBooleanType XROIImage(Display *display,
             windows->image.highlight_context,&highlight_info);
         }
       else
-        ifIfMagickTrue(windows->info.mapped) )
+        if (IfMagickTrue(windows->info.mapped) )
           (void) XWithdrawWindow(display,windows->info.id,windows->info.screen);
       /*
         Wait for next event.
@@ -11593,7 +11593,7 @@ static MagickBooleanType XROIImage(Display *display,
     (void) XMapWindow(display,windows->info.id);
     do
     {
-      ifIfMagickTrue(windows->info.mapped) )
+      if (IfMagickTrue(windows->info.mapped) )
         {
           /*
             Display pointer position.
@@ -11915,7 +11915,7 @@ static MagickBooleanType XROIImage(Display *display,
           */
           x=event.xmotion.x;
           y=event.xmotion.y;
-          ifIfMagickTrue(windows->info.mapped) )
+          if (IfMagickTrue(windows->info.mapped) )
             {
               if ((x < (int) (windows->info.x+windows->info.width)) &&
                   (y < (int) (windows->info.y+windows->info.height)))
@@ -12315,7 +12315,7 @@ static MagickBooleanType XRotateImage(Display *display,
             /*
               Display info and draw rotation line.
             */
-            ifIfMagickFalse(windows->info.mapped) )
+            if (IfMagickFalse(windows->info.mapped) )
               (void) XMapWindow(display,windows->info.id);
             (void) FormatLocaleString(text,MaxTextExtent," %g",
               direction == VerticalRotateCommand ? degrees-90.0 : degrees);
@@ -12324,7 +12324,7 @@ static MagickBooleanType XRotateImage(Display *display,
               windows->image.highlight_context,&rotate_info);
           }
         else
-          ifIfMagickTrue(windows->info.mapped) )
+          if (IfMagickTrue(windows->info.mapped) )
             (void) XWithdrawWindow(display,windows->info.id,
               windows->info.screen);
         /*
@@ -12459,7 +12459,7 @@ static MagickBooleanType XRotateImage(Display *display,
         }
       }
     }
-  ifIfMagickTrue(windows->image.orphan) )
+  if (IfMagickTrue(windows->image.orphan) )
     return(MagickTrue);
   if (normalized_degrees != 0.0)
     {
@@ -12570,7 +12570,7 @@ static MagickBooleanType XSaveImage(Display *display,
   XFileBrowserWidget(display,windows,"Save",filename);
   if (*filename == '\0')
     return(MagickTrue);
-  ifIfMagickTrue(IsPathAccessible(filename)) )
+  if (IfMagickTrue(IsPathAccessible(filename)) )
     {
       int
         status;
@@ -12644,7 +12644,7 @@ static MagickBooleanType XSaveImage(Display *display,
   */
   (void) CopyMagickString(save_image->filename,filename,MaxTextExtent);
   status=WriteImage(image_info,save_image,exception);
-  ifIfMagickTrue(status) )
+  if (IfMagickTrue(status) )
     image->taint=MagickFalse;
   save_image=DestroyImage(save_image);
   image_info=DestroyImageInfo(image_info);
@@ -12759,7 +12759,7 @@ static void XScreenEvent(Display *display,XWindows *windows,XEvent *event,
              if (y >= (int) windows->image.height)
                y=(int) (windows->image.height-1);
             windows->magnify.y=windows->image.y+y;
-            ifIfMagickFalse(windows->magnify.mapped) )
+            if (IfMagickFalse(windows->magnify.mapped) )
               (void) XMapRaised(display,windows->magnify.id);
             XMakeMagnifyImage(display,windows,exception);
             if (event->type == ButtonRelease)
@@ -12798,7 +12798,7 @@ static void XScreenEvent(Display *display,XWindows *windows,XEvent *event,
           */
           windows->magnify.width=(unsigned int) event->xconfigure.width;
           windows->magnify.height=(unsigned int) event->xconfigure.height;
-          ifIfMagickFalse(windows->magnify.mapped) )
+          if (IfMagickFalse(windows->magnify.mapped) )
             break;
           magnify=1;
           while ((int) magnify <= event->xconfigure.width)
@@ -12883,7 +12883,7 @@ static void XScreenEvent(Display *display,XWindows *windows,XEvent *event,
     {
       while (XCheckMaskEvent(display,ButtonMotionMask,event)) ;
       if (event->xmotion.window == windows->image.id)
-        ifIfMagickTrue(windows->magnify.mapped) )
+        if (IfMagickTrue(windows->magnify.mapped) )
           {
             /*
               Update magnified image.
@@ -12974,7 +12974,7 @@ static void XSetCropGeometry(Display *display,XWindows *windows,
     height,
     width;
 
-  ifIfMagickTrue(windows->info.mapped) )
+  if (IfMagickTrue(windows->info.mapped) )
     {
       /*
         Display info on cropping rectangle.
@@ -13212,7 +13212,7 @@ static Image *XTileImage(Display *display,XResourceInfo *resource_info,
       /*
         Delete tile image.
       */
-      ifIfMagickFalse(IsPathAccessible(filename)) )
+      if (IfMagickFalse(IsPathAccessible(filename)) )
         {
           XNoticeWidget(display,windows,"Image file does not exist:",filename);
           break;
@@ -13220,8 +13220,8 @@ static Image *XTileImage(Display *display,XResourceInfo *resource_info,
       status=XConfirmWidget(display,windows,"Really delete tile",filename);
       if (status <= 0)
         break;
-      status=IsMagickTrue(remove_utf8(filename));
-      ifIfMagickTrue(status) )
+      status=ShredFile(filename);
+      if (IfMagickTrue(status) )
         {
           XNoticeWidget(display,windows,"Unable to delete image file:",
             filename);
@@ -13258,7 +13258,7 @@ static Image *XTileImage(Display *display,XResourceInfo *resource_info,
           q++;
         (void) CopyMagickString(filename,p,(size_t) (q-p+1));
         p=q;
-        ifIfMagickTrue(IsPathAccessible(filename)) )
+        if (IfMagickTrue(IsPathAccessible(filename)) )
           {
             tile++;
             continue;
@@ -13281,7 +13281,7 @@ static Image *XTileImage(Display *display,XResourceInfo *resource_info,
             SetPixelInfoPixel(image,&pixel,s);
             s+=GetPixelChannels(image);
           }
-          ifIfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
+          if (IfMagickFalse(SyncCacheViewAuthenticPixels(image_view,exception)) )
             break;
         }
         image_view=DestroyCacheView(image_view);
@@ -13652,7 +13652,7 @@ static Image *XVisualDirectoryImage(Display *display,
   number_files=1;
   filelist[0]=filenames;
   status=ExpandFilenames(&number_files,&filelist);
-  ifIfMagickFalse(status) || (number_files == 0))
+  if (IfMagickFalse(status) || (number_files == 0))
     {
       if (number_files == 0)
         ThrowXWindowFatalException(ImageError,"NoImagesWereFound",filenames)
@@ -13717,7 +13717,7 @@ static Image *XVisualDirectoryImage(Display *display,
 
             proceed=SetImageProgress(images,LoadImageTag,(MagickOffsetType) i,
               (MagickSizeType) number_files);
-            ifIfMagickFalse(proceed) )
+            if (IfMagickFalse(proceed) )
               break;
           }
       }
@@ -13828,7 +13828,7 @@ MagickExport MagickBooleanType XDisplayBackgroundImage(Display *display,
   */
   assert(image != (Image *) NULL);
   assert(image->signature == MagickSignature);
-  ifIfMagickTrue(image->debug) )
+  if (IfMagickTrue(image->debug) )
     (void) LogMagickEvent(TraceEvent,GetMagickModule(),"%s",image->filename);
   resources=(*resource_info);
   window_info.id=(Window) NULL;
@@ -13938,12 +13938,12 @@ MagickExport MagickBooleanType XDisplayBackgroundImage(Display *display,
   window_info.y=(int) geometry_info.y;
   status=XMakeImage(display,&resources,&window_info,image,window_info.width,
     window_info.height,exception);
-  ifIfMagickFalse(status) )
+  if (IfMagickFalse(status) )
     ThrowXWindowFatalException(XServerFatalError,"UnableToCreateXImage",
       image->filename);
   window_info.x=0;
   window_info.y=0;
-  ifIfMagickTrue(image->debug) )
+  if (IfMagickTrue(image->debug) )
     {
       (void) LogMagickEvent(X11Event,GetMagickModule(),
         "Image: %s[%.20g] %.20gx%.20g ",image->filename,(double) image->scene,
@@ -13958,7 +13958,7 @@ MagickExport MagickBooleanType XDisplayBackgroundImage(Display *display,
   */
   width=(int) window_info.width;
   height=(int) window_info.height;
-  ifIfMagickTrue(resources.backdrop) )
+  if (IfMagickTrue(resources.backdrop) )
     {
       /*
         Center image on window.
@@ -14523,7 +14523,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
 
   assert(image != (Image **) NULL);
   assert((*image)->signature == MagickSignature);
-  ifIfMagickTrue((*image)->debug) )
+  if (IfMagickTrue((*image)->debug) )
     (void) LogMagickEvent(TraceEvent,GetMagickModule(),"%s",(*image)->filename);
   display_image=(*image);
   warning_handler=(WarningHandler) NULL;
@@ -14596,7 +14596,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   manager_hints=windows->manager_hints;
   root_window=XRootWindow(display,visual_info->screen);
   nexus=NewImageList();
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     {
       (void) LogMagickEvent(X11Event,GetMagickModule(),
         "Image: %s[%.20g] %.20gx%.20g ",display_image->filename,
@@ -14625,7 +14625,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   manager_hints->initial_state=WithdrawnState;
   XMakeWindow(display,root_window,argv,argc,class_hints,manager_hints,
     &windows->context);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),
       "Window id: 0x%lx (context)",windows->context.id);
   context_values.background=pixel->background_color.pixel;
@@ -14676,7 +14676,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   manager_hints->initial_state=IconicState;
   XMakeWindow(display,root_window,argv,argc,class_hints,manager_hints,
     &windows->icon);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),"Window id: 0x%lx (icon)",
       windows->icon.id);
   /*
@@ -14698,7 +14698,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   XGetWindowInfo(display,visual_info,map_info,pixel,font_info,resource_info,
     &windows->image);
   windows->image.shape=MagickTrue;  /* non-rectangular shape hint */
-  ifIfMagickFalse(resource_info->use_shared_memory) )
+  if (IfMagickFalse(resource_info->use_shared_memory) )
     windows->image.shared_memory=MagickFalse;
   if ((resource_info->title != (char *) NULL) && !(*state & MontageImageState))
     {
@@ -14777,7 +14777,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
         NormalState;
       XMakeWindow(display,root_window,argv,argc,class_hints,manager_hints,
         &windows->backdrop);
-      ifIfMagickTrue(display_image->debug) )
+      if (IfMagickTrue(display_image->debug) )
         (void) LogMagickEvent(X11Event,GetMagickModule(),
           "Window id: 0x%lx (backdrop)",windows->backdrop.id);
       (void) XMapWindow(display,windows->backdrop.id);
@@ -14809,7 +14809,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       manager_hints->flags|=WindowGroupHint;
       manager_hints->window_group=windows->group_leader.id;
       (void) XSelectInput(display,windows->group_leader.id,StructureNotifyMask);
-      ifIfMagickTrue(display_image->debug) )
+      if (IfMagickTrue(display_image->debug) )
         (void) LogMagickEvent(X11Event,GetMagickModule(),
           "Window id: 0x%lx (group leader)",windows->group_leader.id);
     }
@@ -14821,7 +14821,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   if (windows->group_leader.id != (Window) NULL)
     (void) XSetTransientForHint(display,windows->image.id,
       windows->group_leader.id);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),"Window id: 0x%lx (image)",
       windows->image.id);
   /*
@@ -14849,9 +14849,9 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   windows->info.shadow_stipple=XCreateBitmapFromData(display,
     windows->info.id,(char *) ShadowBitmap,ShadowWidth,ShadowHeight);
   (void) XSetTransientForHint(display,windows->info.id,windows->image.id);
-  ifIfMagickTrue(windows->image.mapped) )
+  if (IfMagickTrue(windows->image.mapped) )
     (void) XWithdrawWindow(display,windows->info.id,windows->info.screen);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),"Window id: 0x%lx (info)",
       windows->info.id);
   /*
@@ -14883,9 +14883,9 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   windows->command.shadow_stipple=XCreateBitmapFromData(display,
     windows->command.id,(char *) ShadowBitmap,ShadowWidth,ShadowHeight);
   (void) XSetTransientForHint(display,windows->command.id,windows->image.id);
-  ifIfMagickTrue(windows->command.mapped) )
+  if (IfMagickTrue(windows->command.mapped) )
     (void) XMapRaised(display,windows->command.id);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),
       "Window id: 0x%lx (command)",windows->command.id);
   /*
@@ -14914,7 +14914,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   windows->widget.shadow_stipple=XCreateBitmapFromData(display,
     windows->widget.id,(char *) ShadowBitmap,ShadowWidth,ShadowHeight);
   (void) XSetTransientForHint(display,windows->widget.id,windows->image.id);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),
       "Window id: 0x%lx (widget)",windows->widget.id);
   /*
@@ -14938,7 +14938,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   windows->popup.shadow_stipple=XCreateBitmapFromData(display,
     windows->popup.id,(char *) ShadowBitmap,ShadowWidth,ShadowHeight);
   (void) XSetTransientForHint(display,windows->popup.id,windows->image.id);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),
       "Window id: 0x%lx (pop up)",windows->popup.id);
   /*
@@ -14946,7 +14946,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   */
   XGetWindowInfo(display,visual_info,map_info,pixel,font_info,
     resource_info,&windows->magnify);
-  ifIfMagickFalse(resource_info->use_shared_memory) )
+  if (IfMagickFalse(resource_info->use_shared_memory) )
     windows->magnify.shared_memory=MagickFalse;
   (void) FormatLocaleString(resource_name,MaxTextExtent,"%s.magnify",
     resource_info->client_name);
@@ -14980,7 +14980,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   manager_hints->window_group=windows->image.id;
   XMakeWindow(display,root_window,argv,argc,class_hints,manager_hints,
     &windows->magnify);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),
       "Window id: 0x%lx (magnify)",windows->magnify.id);
   (void) XSetTransientForHint(display,windows->magnify.id,windows->image.id);
@@ -15009,13 +15009,13 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   manager_hints->window_group=windows->image.id;
   XMakeWindow(display,root_window,argv,argc,class_hints,manager_hints,
     &windows->pan);
-  ifIfMagickTrue(display_image->debug) )
+  if (IfMagickTrue(display_image->debug) )
     (void) LogMagickEvent(X11Event,GetMagickModule(),"Window id: 0x%lx (pan)",
       windows->pan.id);
   (void) XSetTransientForHint(display,windows->pan.id,windows->image.id);
-  ifIfMagickTrue(windows->info.mapped) )
+  if (IfMagickTrue(windows->info.mapped) )
     (void) XWithdrawWindow(display,windows->info.id,windows->info.screen);
-  ifIfMagickFalse(windows->image.mapped) ||
+  if (IfMagickFalse(windows->image.mapped) ||
       (windows->backdrop.id != (Window) NULL))
     (void) XMapWindow(display,windows->image.id);
   /*
@@ -15041,17 +15041,17 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       display_image->filename);
   status=XMakeImage(display,resource_info,&windows->image,display_image,
     width,height,exception);
-  ifIfMagickFalse(status) )
+  if (IfMagickFalse(status) )
     ThrowXWindowFatalException(XServerFatalError,"UnableToCreateXImage",
       display_image->filename);
   status=XMakeImage(display,resource_info,&windows->magnify,(Image *) NULL,
     windows->magnify.width,windows->magnify.height,exception);
-  ifIfMagickFalse(status) )
+  if (IfMagickFalse(status) )
     ThrowXWindowFatalException(XServerFatalError,"UnableToCreateXImage",
       display_image->filename);
-  ifIfMagickTrue(windows->magnify.mapped) )
+  if (IfMagickTrue(windows->magnify.mapped) )
     (void) XMapRaised(display,windows->magnify.id);
-  ifIfMagickTrue(windows->pan.mapped) )
+  if (IfMagickTrue(windows->pan.mapped) )
     (void) XMapRaised(display,windows->pan.id);
   windows->image.window_changes.width=(int) display_image->columns;
   windows->image.window_changes.height=(int) display_image->rows;
@@ -15064,7 +15064,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
   delay=display_image->delay/MagickMax(display_image->ticks_per_second,1L);
   timer=time((time_t *) NULL)+(delay == 0 ? 1 : delay)+1;
   update_time=0;
-  ifIfMagickTrue(resource_info->update) )
+  if (IfMagickTrue(resource_info->update) )
     {
       MagickBooleanType
         status;
@@ -15073,7 +15073,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
         Determine when file data was last modified.
       */
       status=GetPathAttributes(display_image->filename,&attributes);
-      ifIfMagickTrue(status) )
+      if (IfMagickTrue(status) )
         update_time=attributes.st_mtime;
     }
   *state&=(~FormerImageState);
@@ -15084,12 +15084,12 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
     /*
       Handle a window event.
     */
-    ifIfMagickTrue(windows->image.mapped) )
+    if (IfMagickTrue(windows->image.mapped) )
       if ((display_image->delay != 0) || (resource_info->update != 0))
         {
           if (timer < time((time_t *) NULL))
             {
-              ifIfMagickFalse(resource_info->update) )
+              if (IfMagickFalse(resource_info->update) )
                 *state|=NextImageState | ExitState;
               else
                 {
@@ -15100,7 +15100,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                     Determine if image file was modified.
                   */
                   status=GetPathAttributes(display_image->filename,&attributes);
-                  ifIfMagickTrue(status) )
+                  if (IfMagickTrue(status) )
                     if (update_time != attributes.st_mtime)
                       {
                         /*
@@ -15133,9 +15133,9 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
         }
     timestamp=time((time_t *) NULL);
     (void) XNextEvent(display,&event);
-    ifIfMagickFalse(windows->image.stasis) )
+    if (IfMagickFalse(windows->image.stasis) )
       windows->image.stasis=IsMagickTrue((time((time_t *) NULL)-timestamp) > 0);
-    ifIfMagickFalse(windows->magnify.stasis) )
+    if (IfMagickFalse(windows->magnify.stasis) )
       windows->magnify.stasis=IsMagickTrue((time((time_t *) NULL)-timestamp) > 0);
     if (event.xany.window == windows->command.id)
       {
@@ -15168,7 +15168,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
     {
       case ButtonPress:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Button Press: 0x%lx %u +%d+%d",event.xbutton.window,
             event.xbutton.button,event.xbutton.x,event.xbutton.y);
@@ -15208,7 +15208,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                 /*
                   Map/unmap Command widget.
                 */
-                ifIfMagickTrue(windows->command.mapped) )
+                if (IfMagickTrue(windows->command.mapped) )
                   (void) XWithdrawWindow(display,windows->command.id,
                     windows->command.screen);
                 else
@@ -15363,7 +15363,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       case ButtonRelease:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Button Release: 0x%lx %u +%d+%d",event.xbutton.window,
             event.xbutton.button,event.xbutton.x,event.xbutton.y);
@@ -15371,7 +15371,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       case ClientMessage:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Client Message: 0x%lx 0x%lx %d 0x%lx",event.xclient.window,
             event.xclient.message_type,event.xclient.format,(unsigned long)
@@ -15417,7 +15417,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                     (unsigned long) magick_windows[i]->mask,
                     &magick_windows[i]->attributes);
                 }
-                ifIfMagickTrue(windows->pan.mapped) )
+                if (IfMagickTrue(windows->pan.mapped) )
                   {
                     (void) XSetWindowBackgroundPixmap(display,windows->pan.id,
                       windows->pan.pixmap);
@@ -15534,7 +15534,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       case ConfigureNotify:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Configure Notify: 0x%lx %dx%d+%d+%d %d",event.xconfigure.window,
             event.xconfigure.width,event.xconfigure.height,event.xconfigure.x,
@@ -15553,7 +15553,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                   Position the transient windows relative of the Image window.
                 */
                 if (windows->command.geometry == (char *) NULL)
-                  ifIfMagickFalse(windows->command.mapped) )
+                  if (IfMagickFalse(windows->command.mapped) )
                     {
                       windows->command.x=event.xconfigure.x-
                         windows->command.width-25;
@@ -15566,7 +15566,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                         &window_changes);
                     }
                 if (windows->widget.geometry == (char *) NULL)
-                  ifIfMagickFalse(windows->widget.mapped) )
+                  if (IfMagickFalse(windows->widget.mapped) )
                     {
                       windows->widget.x=event.xconfigure.x+
                         event.xconfigure.width/10;
@@ -15580,7 +15580,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                         &window_changes);
                     }
                 if (windows->magnify.geometry == (char *) NULL)
-                  ifIfMagickFalse(windows->magnify.mapped) )
+                  if (IfMagickFalse(windows->magnify.mapped) )
                     {
                       windows->magnify.x=event.xconfigure.x+
                         event.xconfigure.width+25;
@@ -15593,7 +15593,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                         &window_changes);
                     }
                 if (windows->pan.geometry == (char *) NULL)
-                  ifIfMagickFalse(windows->pan.mapped) )
+                  if (IfMagickFalse(windows->pan.mapped) )
                     {
                       windows->pan.x=event.xconfigure.x+
                         event.xconfigure.width+25;
@@ -15619,7 +15619,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                 windows->image.x=vid_info.x;
                 windows->image.y=vid_info.y;
               }
-            ifIfMagickTrue(windows->image.mapped) &&
+            if (IfMagickTrue(windows->image.mapped) &&
                 IfMagickTrue(windows->image.stasis) )
               {
                 /*
@@ -15640,7 +15640,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                 XDrawPanRectangle(display,windows);
               }
             else
-              ifIfMagickTrue(windows->pan.mapped) )
+              if (IfMagickTrue(windows->pan.mapped) )
                 (void) XWithdrawWindow(display,windows->pan.id,
                   windows->pan.screen);
             break;
@@ -15655,7 +15655,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
             */
             windows->magnify.width=(unsigned int) event.xconfigure.width;
             windows->magnify.height=(unsigned int) event.xconfigure.height;
-            ifIfMagickFalse(windows->magnify.mapped) )
+            if (IfMagickFalse(windows->magnify.mapped) )
               break;
             magnify=1;
             while ((int) magnify <= event.xconfigure.width)
@@ -15673,7 +15673,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
                   &window_changes);
                 break;
               }
-            ifIfMagickTrue(windows->magnify.mapped) &&
+            if (IfMagickTrue(windows->magnify.mapped) &&
                 IfMagickTrue(windows->magnify.stasis) )
               {
                 status=XMakeImage(display,resource_info,&windows->magnify,
@@ -15683,7 +15683,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
               }
             break;
           }
-        ifIfMagickTrue(windows->magnify.mapped) &&
+        if (IfMagickTrue(windows->magnify.mapped) &&
             (event.xconfigure.window == windows->pan.id))
           {
             /*
@@ -15714,7 +15714,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
         /*
           Group leader has exited.
         */
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Destroy Notify: 0x%lx",event.xdestroywindow.window);
         if (event.xdestroywindow.window == windows->group_leader.id)
@@ -15736,7 +15736,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       case Expose:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Expose: 0x%lx %dx%d+%d+%d",event.xexpose.window,
             event.xexpose.width,event.xexpose.height,event.xexpose.x,
@@ -15782,7 +15782,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
         length=XLookupString((XKeyEvent *) &event.xkey,command,(int)
           sizeof(command),&key_symbol,(XComposeStatus *) NULL);
         *(command+length)='\0';
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Key press: %d 0x%lx (%s)",event.xkey.state,(unsigned long)
             key_symbol,command);
@@ -15821,7 +15821,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
         */
         (void) XLookupString((XKeyEvent *) &event.xkey,command,(int)
           sizeof(command),&key_symbol,(XComposeStatus *) NULL);
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Key release: 0x%lx (%c)",(unsigned long) key_symbol,*command);
         break;
@@ -15838,7 +15838,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       case MapNotify:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),"Map Notify: 0x%lx",
             event.xmap.window);
         if (event.xmap.window == windows->backdrop.id)
@@ -15946,7 +15946,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
           after,
           length;
 
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Property Notify: 0x%lx 0x%lx %d",event.xproperty.window,
             event.xproperty.atom,event.xproperty.state);
@@ -15978,7 +15978,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       case ReparentNotify:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Reparent Notify: 0x%lx=>0x%lx",event.xreparent.parent,
             event.xreparent.window);
@@ -15986,7 +15986,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       case UnmapNotify:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),
             "Unmap Notify: 0x%lx",event.xunmap.window);
         if (event.xunmap.window == windows->backdrop.id)
@@ -16049,7 +16049,7 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       }
       default:
       {
-        ifIfMagickTrue(display_image->debug) )
+        if (IfMagickTrue(display_image->debug) )
           (void) LogMagickEvent(X11Event,GetMagickModule(),"Event type: %d",
             event.type);
         break;
@@ -16060,12 +16060,12 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
     (void) XMagickCommand(display,resource_info,windows,FreeBuffersCommand,
       &display_image,exception);
   else
-    ifIfMagickTrue(resource_info->confirm_edit) )
+    if (IfMagickTrue(resource_info->confirm_edit) )
       {
         /*
           Query user if image has changed.
         */
-        ifIfMagickFalse(resource_info->immutable) &&
+        if (IfMagickFalse(resource_info->immutable) &&
             IfMagickTrue(display_image->taint))
           {
             int
@@ -16088,18 +16088,18 @@ MagickExport Image *XDisplayImage(Display *display,XResourceInfo *resource_info,
       /*
         Withdraw pan and Magnify window.
       */
-      ifIfMagickTrue(windows->info.mapped) )
+      if (IfMagickTrue(windows->info.mapped) )
         (void) XWithdrawWindow(display,windows->info.id,windows->info.screen);
-      ifIfMagickTrue(windows->magnify.mapped) )
+      if (IfMagickTrue(windows->magnify.mapped) )
         (void) XWithdrawWindow(display,windows->magnify.id,
           windows->magnify.screen);
-      ifIfMagickTrue(windows->command.mapped) )
+      if (IfMagickTrue(windows->command.mapped) )
         (void) XWithdrawWindow(display,windows->command.id,
           windows->command.screen);
     }
-  ifIfMagickTrue(windows->pan.mapped) )
+  if (IfMagickTrue(windows->pan.mapped) )
     (void) XWithdrawWindow(display,windows->pan.id,windows->pan.screen);
-  ifIfMagickFalse(resource_info->backdrop) )
+  if (IfMagickFalse(resource_info->backdrop) )
     if (windows->backdrop.mapped)
       {
         (void) XWithdrawWindow(display,windows->backdrop.id,
@@ -16196,7 +16196,7 @@ MagickExport MagickBooleanType DisplayImages(const ImageInfo *image_info,
   assert(image_info->signature == MagickSignature);
   assert(image != (Image *) NULL);
   assert(image->signature == MagickSignature);
-  ifIfMagickTrue(image->debug) )
+  if (IfMagickTrue(image->debug) )
     (void) LogMagickEvent(TraceEvent,GetMagickModule(),"%s",image->filename);
   (void) ThrowMagickException(exception,GetMagickModule(),MissingDelegateError,
     "DelegateLibrarySupportNotBuiltIn","'%s' (X11)",image->filename);
index 449c1c0a120463a5d8932a75ab44d6c24339683f..4cdf9f8fcac15b8e4a204067f2a801380d4de28a 100644 (file)
@@ -460,7 +460,7 @@ static StringInfo *GenerateEntropicChaos(RandomInfo *random_info)
 #endif
     if (file != -1)
       (void) close(file);
-    (void) remove_utf8(filename);
+    (void) ShredFile(filename);
     SetStringInfoLength(chaos,strlen(filename));
     SetStringInfoDatum(chaos,(unsigned char *) filename);
     ConcatenateStringInfo(entropy,chaos);
index 4ee9258788dfaa4453f3dbf9cbde3b5834e92409..977465775afe509c2179ddd3e6c66b6c42a655bf 100644 (file)
@@ -315,7 +315,7 @@ MagickPrivate void AsynchronousResourceComponentTerminus(void)
   path=(const char *) GetNextKeyInSplayTree(temporary_resources);
   while (path != (const char *) NULL)
   {
-    (void) remove_utf8(path);
+    (void) ShredFile(path);
     path=(const char *) GetNextKeyInSplayTree(temporary_resources);
   }
 }
@@ -347,7 +347,7 @@ MagickPrivate void AsynchronousResourceComponentTerminus(void)
 
 static void *DestroyTemporaryResources(void *temporary_resource)
 {
-  (void) remove_utf8((char *) temporary_resource);
+  (void) ShredFile((char *) temporary_resource);
   temporary_resource=DestroyString((char *) temporary_resource);
   return((void *) NULL);
 }
@@ -900,8 +900,8 @@ MagickExport MagickBooleanType RelinquishUniqueFileResource(const char *path)
     }
   (void) CopyMagickString(cache_path,path,MaxTextExtent);
   AppendImageFormat("cache",cache_path);
-  (void) remove_utf8(cache_path);
-  return(remove_utf8(path) == 0 ? MagickTrue : MagickFalse);
+  (void) ShredFile(cache_path);
+  return(ShredFile(path));
 }
 \f
 /*
index ed2a953fa7c9384cf807e4311ca27447c0c2ebea..e7a178c4397af762fd6df2fbbf308d48ce69dfe0 100644 (file)
@@ -31,7 +31,8 @@ extern MagickPrivate char
   **ListFiles(const char *,const char *,size_t *);
 
 extern MagickPrivate MagickBooleanType
-  GetExecutionPath(char *,const size_t);
+  GetExecutionPath(char *,const size_t),
+  ShredFile(const char *);
 
 extern MagickPrivate ssize_t
   GetMagickPageSize(void);
index 3bec2ab0130183c479e3b0024e0aca432cb39cec..50155e4718bd142a0be5fed596d3840da32dc6f4 100644 (file)
@@ -1412,7 +1412,7 @@ MagickPrivate char **GetPathComponents(const char *path,
 %
 %  The format of the IsPathAccessible method is:
 %
-%      MagickBooleanType IsPathAccessible(const char *filename)
+%      MagickBooleanType IsPathAccessible(const char *path)
 %
 %  A description of each parameter follows.
 %
@@ -1755,6 +1755,43 @@ MagickExport size_t MultilineCensus(const char *label)
 %                                                                             %
 %                                                                             %
 %                                                                             %
+%   S h r e a d F i l e                                                       %
+%                                                                             %
+%                                                                             %
+%                                                                             %
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%
+%  ShredFile() first overwrites the specified file with random data and then
+%  removed it.  The overwrite is optional and is only required to help keep
+%  the contents of the file private.
+%
+%  The format of the ShredFile method is:
+%
+%      MagickBooleanType ShredFile(const char *path)
+%
+%  A description of each parameter follows.
+%
+%    o path:  Specifies a path to a file.
+%
+*/
+MagickPrivate MagickBooleanType ShredFile(const char *path)
+{
+  int
+    status;
+
+  if ((path == (const char *) NULL) || (*path == '\0'))
+    return(MagickFalse);
+  status=remove_utf8(path);
+  if (status == -1)
+    return(MagickFalse);
+  return(MagickTrue);
+}
+\f
+/*
+%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
+%                                                                             %
+%                                                                             %
+%                                                                             %
 %   S y s t e m C o m m a n d                                                 %
 %                                                                             %
 %                                                                             %