com php-src: Fix #72604: imagearc() ignores thickness for full arcs: NEWS ext/gd/libgd/gd.c ext/gd/tests/bug72604.phpt
| From: | Christoph Michael Becker | Date: | Fri, 15 Jul 2016 17:45:52 +0000 |
| Subject: | com php-src: Fix #72604: imagearc() ignores thickness for full arcs: NEWS ext/gd/libgd/gd.c ext/gd/tests/bug72604.phpt | ||
| Groups: | php.cvs | ||
| Request: | Send a blank email to php-cvs+get-93037@lists.php.net to get a copy of this message | ||
Commit: 3666cfab973c5aba86d0562d14de8ed74e66cdb7
Author: Christoph M. Becker <cmb@php.net> Fri, 15 Jul 2016 19:42:26 +0200
Parents: a4aa4f9772a6c30f69db8560cde1f5fe4545b174
Branches: PHP-5.6 PHP-7.0 master
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=3666cfab973c5aba86d0562d14de8ed74e66cdb7
Log:
Fix #72604: imagearc() ignores thickness for full arcs
We remove the special casing for full arcs, what conforms to external libgd.
Bugs:
https://bugs.php.net/72604
Changed paths:
M NEWS
M ext/gd/libgd/gd.c
A ext/gd/tests/bug72604.phpt
Diff:
diff --git a/NEWS b/NEWS
index 455e44b..caf8073 100644
--- a/NEWS
+++ b/NEWS
@@ -29,6 +29,7 @@ PHP NEWS
- GD:
. Fixed bug #72596 (imagetypes function won't advertise WEBP support). (cmb)
+ . Fixed bug #72604 (imagearc() ignores thickness for full arcs). (cmb)
- Intl:
. Partially fixed #72506 (idn_to_ascii for UTS #46 incorrect for long domain
diff --git a/ext/gd/libgd/gd.c b/ext/gd/libgd/gd.c
index a5799c5..3b5d829 100644
--- a/ext/gd/libgd/gd.c
+++ b/ext/gd/libgd/gd.c
@@ -1673,11 +1673,7 @@ long lsqrt (long n)
void gdImageArc (gdImagePtr im, int cx, int cy, int w, int h, int s, int e, int color)
{
- if ((s % 360) == (e % 360)) {
- gdImageEllipse(im, cx, cy, w, h, color);
- } else {
- gdImageFilledArc(im, cx, cy, w, h, s, e, color, gdNoFill);
- }
+ gdImageFilledArc(im, cx, cy, w, h, s, e, color, gdNoFill);
}
void gdImageFilledArc (gdImagePtr im, int cx, int cy, int w, int h, int s, int e, int color, int
style)
diff --git a/ext/gd/tests/bug72604.phpt b/ext/gd/tests/bug72604.phpt
new file mode 100644
index 0000000..b1dea5e
--- /dev/null
+++ b/ext/gd/tests/bug72604.phpt
@@ -0,0 +1,19 @@
+--TEST--
+Bug #72604 (imagearc() ignores thickness for full arcs)
+--SKIPIF--
+<?php
+if (!extension_loaded('gd')) die('skip requires ext/gd');
+if (!(imagetypes() & IMG_PNG)) die('skip requires PNG support');
+?>
+--FILE--
+<?php
+$im = imagecreatetruecolor(100, 100);
+imagesetthickness($im, 5);
+imagearc($im, 50, 50, 90, 90, 0, 360, 0xffffff);
+ob_start();
+imagepng($im);
+$imagestring = ob_get_clean();
+echo md5($imagestring);
+?>
+--EXPECT--
+2ffa6afb915afbdf870cf6459477bc8a