GraphicsMagick: ReadXPMImage(): Eliminate unnecessary strlen() u...

GraphicsMagick Commits <[email protected]>
Newsgroups gmane.comp.video.graphicsmagick.cvs
Message-ID <mailman.10599.1666997962.1350.graphicsmagick-commit@lists.sourceforge.net>
changeset d6f7fda3716b in /hg/GraphicsMagick
details: http://hg.GraphicsMagick.org/hg/GraphicsMagick?cmd=changeset;node=d6f7fda3716b
summary: ReadXPMImage(): Eliminate unnecessary strlen() upon memory reallocation for better performance.

diffstat:

 ChangeLog                              |   5 +++++
 VisualMagick/installer/inc/version.isx |   4 ++--
 coders/xpm.c                           |   7 +++++--
 magick/version.h                       |   4 ++--
 www/Changelog.html                     |  27 +++++++++++++++++++++++++++
 5 files changed, 41 insertions(+), 6 deletions(-)

diffs (91 lines):

diff -r 94516c1cc211 -r d6f7fda3716b ChangeLog
--- a/ChangeLog	Sat Oct 29 00:09:43 2022 +0200
+++ b/ChangeLog	Fri Oct 28 17:59:05 2022 -0500
@@ -1,3 +1,8 @@
+2022-10-28  Bob Friesenhahn  <[email protected]>
+
+	* coders/xpm.c (ReadXPMImage): Eliminate unnecessary strlen() upon
+	memory reallocation for better performance.
+
 2022-10-29  Fojtik Jaroslav  <[email protected]>
 
 	* PerlMagick/t/input_bilevel.xpm
diff -r 94516c1cc211 -r d6f7fda3716b VisualMagick/installer/inc/version.isx
--- a/VisualMagick/installer/inc/version.isx	Sat Oct 29 00:09:43 2022 +0200
+++ b/VisualMagick/installer/inc/version.isx	Fri Oct 28 17:59:05 2022 -0500
@@ -10,5 +10,5 @@
 
 #define public MagickPackageName "GraphicsMagick"
 #define public MagickPackageVersion "1.4"
-#define public MagickPackageVersionAddendum ".020221023"
-#define public MagickPackageReleaseDate "snapshot-20221023"
+#define public MagickPackageVersionAddendum ".020221028"
+#define public MagickPackageReleaseDate "snapshot-20221028"
diff -r 94516c1cc211 -r d6f7fda3716b coders/xpm.c
--- a/coders/xpm.c	Sat Oct 29 00:09:43 2022 +0200
+++ b/coders/xpm.c	Fri Oct 28 17:59:05 2022 -0500
@@ -320,8 +320,11 @@
               MagickFreeResourceLimitedMemory(xpm_buffer);
               break;
             }
-          xpm_buffer=new_xpm_buffer;
-          p=xpm_buffer+strlen(xpm_buffer);
+          if (xpm_buffer != new_xpm_buffer)
+            {
+              p=(p-xpm_buffer)+new_xpm_buffer;
+              xpm_buffer=new_xpm_buffer;
+            }
         }
     }
   if (xpm_buffer == (char *) NULL)
diff -r 94516c1cc211 -r d6f7fda3716b magick/version.h
--- a/magick/version.h	Sat Oct 29 00:09:43 2022 +0200
+++ b/magick/version.h	Fri Oct 28 17:59:05 2022 -0500
@@ -38,8 +38,8 @@
 #define MagickLibVersion  0x272400
 #define MagickLibVersionText  "1.4"
 #define MagickLibVersionNumber 27,24,0
-#define MagickChangeDate   "20221023"
-#define MagickReleaseDate  "snapshot-20221023"
+#define MagickChangeDate   "20221028"
+#define MagickReleaseDate  "snapshot-20221028"
 
 /*
   The MagickLibInterfaceNewest and MagickLibInterfaceOldest defines
diff -r 94516c1cc211 -r d6f7fda3716b www/Changelog.html
--- a/www/Changelog.html	Sat Oct 29 00:09:43 2022 +0200
+++ b/www/Changelog.html	Fri Oct 28 17:59:05 2022 -0500
@@ -37,6 +37,33 @@
 </div>
 
 <div class="document">
+<p>2022-10-28  Bob Friesenhahn  &lt;<a class="reference external" href="mailto:bfriesen&#37;&#52;&#48;simple&#46;dallas&#46;tx&#46;us">bfriesen<span>&#64;</span>simple<span>&#46;</span>dallas<span>&#46;</span>tx<span>&#46;</span>us</a>&gt;</p>
+<blockquote>
+<ul class="simple">
+<li><p>coders/xpm.c (ReadXPMImage): Eliminate unnecessary strlen() upon
+memory reallocation for better performance.</p></li>
+</ul>
+</blockquote>
+<p>2022-10-29  Fojtik Jaroslav  &lt;<a class="reference external" href="mailto:JaFojtik&#37;&#52;&#48;yandex&#46;com">JaFojtik<span>&#64;</span>yandex<span>&#46;</span>com</a>&gt;</p>
+<blockquote>
+<ul class="simple">
+<li><p>PerlMagick/t/input_bilevel.xpm</p></li>
+<li><p>PerlMagick/t/reference/read/input_xpm_bilevel.miff New testcase
+with bilevel monochrome image.</p></li>
+<li><p>PerlMagick/t/read.t: Added input_xpm_pal16.miff to PerlMagick test
+suite.</p></li>
+</ul>
+</blockquote>
+<p>2022-10-28  Fojtik Jaroslav  &lt;<a class="reference external" href="mailto:JaFojtik&#37;&#52;&#48;yandex&#46;com">JaFojtik<span>&#64;</span>yandex<span>&#46;</span>com</a>&gt;</p>
+<blockquote>
+<ul class="simple">
+<li><p>PerlMagick/t/input_pal16.xpm</p></li>
+<li><p>PerlMagick/t/reference/read/input_xpm_pal16.miff New testcase
+with 16 bit palette.</p></li>
+<li><p>PerlMagick/t/read.t: Added input_xpm_pal16.miff to PerlMagick test
+suite.</p></li>
+</ul>
+</blockquote>
 <p>2022-10-23  Bob Friesenhahn  &lt;<a class="reference external" href="mailto:bfriesen&#37;&#52;&#48;simple&#46;dallas&#46;tx&#46;us">bfriesen<span>&#64;</span>simple<span>&#46;</span>dallas<span>&#46;</span>tx<span>&#46;</span>us</a>&gt;</p>
 <blockquote>
 <ul class="simple">
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.