]> granicus.if.org Git - imagemagick/commitdiff
(no commit message)
authorcristy <urban-warrior@git.imagemagick.org>
Thu, 14 Mar 2013 00:13:29 +0000 (00:13 +0000)
committercristy <urban-warrior@git.imagemagick.org>
Thu, 14 Mar 2013 00:13:29 +0000 (00:13 +0000)
coders/caption.c
coders/label.c

index 2211dcae5357a58248a40fcd9c8c917548cdf83b..3fb2b409d3aa18d4871e1adfbae5701d46ad1ff2 100644 (file)
@@ -155,7 +155,8 @@ static Image *ReadCAPTIONImage(const ImageInfo *image_info,
   if (gravity != (char *) NULL)
     draw_info->gravity=(GravityType) ParseCommandOption(MagickGravityOptions,
       MagickFalse,gravity);
-  if ((*draw_info->text != '\0') && (image_info->pointsize == 0.0))
+  if ((*caption != '\0') && (image->rows != 0) &&
+      (image_info->pointsize == 0.0))
     {
       char
         *text;
@@ -167,27 +168,6 @@ static Image *ReadCAPTIONImage(const ImageInfo *image_info,
       /*
         Auto fit text into bounding box.
       */
-      status=GetMultilineTypeMetrics(image,draw_info,&metrics,exception);
-      if (image->rows == 0)
-        {
-          for ( ; ; )
-          {
-            (void) FormatLocaleString(geometry,MaxTextExtent,"%+g%+g",
-              -metrics.bounds.x1,metrics.ascent);
-            if (draw_info->gravity == UndefinedGravity)
-              (void) CloneString(&draw_info->geometry,geometry);
-            status=GetMultilineTypeMetrics(image,draw_info,&metrics,exception);
-            (void) status;
-            width=(size_t) floor(metrics.width+draw_info->stroke_width+0.5);
-            height=(size_t) floor(metrics.height+draw_info->stroke_width+0.5);
-            if (width > image->columns)
-              break;
-            if (width > (image->columns << 1))
-              break;
-            draw_info->pointsize*=2.0;
-          }
-          image->rows=(size_t) floor(height);
-        }
       for ( ; ; draw_info->pointsize*=2.0)
       {
         text=AcquireString(caption);
@@ -246,7 +226,7 @@ static Image *ReadCAPTIONImage(const ImageInfo *image_info,
         if ((width <= image->columns) && (height <= image->rows))
           break;
       }
-      draw_info->pointsize=floor(draw_info->pointsize);
+      draw_info->pointsize=floor(draw_info->pointsize+0.5);
     }
   i=FormatMagickCaption(image,draw_info,MagickTrue,&metrics,&caption,exception);
   if (image->rows == 0)
index d1cfece79a0a42ce7637de1c3ad7945f8a77e1ef..cfa84703c619697f305ad006cb59793fc4c8f46d 100644 (file)
@@ -130,7 +130,7 @@ static Image *ReadLABELImage(const ImageInfo *image_info,
   label=GetImageProperty(image,"label",exception);
   draw_info=CloneDrawInfo(image_info,(DrawInfo *) NULL);
   draw_info->text=ConstantString(label);
-  if ((*draw_info->text != '\0') && (image_info->pointsize == 0.0))
+  if ((*label != '\0') && (image->rows != 0) && (image_info->pointsize == 0.0))
     {
       double
         high,
@@ -140,27 +140,7 @@ static Image *ReadLABELImage(const ImageInfo *image_info,
         Auto fit text into bounding box.
       */
       status=GetMultilineTypeMetrics(image,draw_info,&metrics,exception);
-      if (image->rows == 0)
-        {
-          for ( ; ; )
-          {
-            (void) FormatLocaleString(geometry,MaxTextExtent,"%+g%+g",
-              -metrics.bounds.x1,metrics.ascent);
-            if (draw_info->gravity == UndefinedGravity)
-              (void) CloneString(&draw_info->geometry,geometry);
-            status=GetMultilineTypeMetrics(image,draw_info,&metrics,exception);
-            (void) status;
-            width=(size_t) floor(metrics.width+draw_info->stroke_width+0.5);
-            height=(size_t) floor(metrics.height+draw_info->stroke_width+0.5);
-            if (width > image->columns)
-              break;
-            if (width > (image->columns << 1))
-              break;
-            draw_info->pointsize*=2.0;
-          }
-          image->rows=(size_t) floor(height);
-        }
-      for ( ; ; )
+      for ( ; ; draw_info->pointsize*=2.0)
       {
         (void) FormatLocaleString(geometry,MaxTextExtent,"%+g%+g",
           -metrics.bounds.x1,metrics.ascent);
@@ -174,7 +154,6 @@ static Image *ReadLABELImage(const ImageInfo *image_info,
           break;
         if ((width > (image->columns << 1)) || (height > (image->rows << 1)))
           break;
-        draw_info->pointsize*=2.0;
       }
       high=draw_info->pointsize/2.0;
       low=high/2.0;
@@ -193,7 +172,7 @@ static Image *ReadLABELImage(const ImageInfo *image_info,
         else
           high=draw_info->pointsize-1.0;
       }
-      for (draw_info->pointsize=(low+high)/2.0; ; )
+      for (draw_info->pointsize=(low+high)/2.0; (high-low) > 1.0; )
       {
         (void) FormatLocaleString(geometry,MaxTextExtent,"%+g%+g",
           -metrics.bounds.x1,metrics.ascent);
@@ -206,7 +185,7 @@ static Image *ReadLABELImage(const ImageInfo *image_info,
           break;
         draw_info->pointsize--;
       }
-      draw_info->pointsize=floor(draw_info->pointsize);
+      draw_info->pointsize=floor(draw_info->pointsize+0.5);
     }
   status=GetMultilineTypeMetrics(image,draw_info,&metrics,exception);
   if (status == MagickFalse)