Hi,
I reverted this commit even if I strongly disagree with how 5.2.4-rc3
has been released and how the 5.2 was frozen without previous mail.
About 5.2.4 in general, there is still issues on windows. I have no
idea how to get the required libs updated (looks like png, freetype
and pslib have not been updated).
If they are updated, we will need another RC. If they are not and/or
if I can't commit fixes then we have serious problems in our release
process and how we communicate. I'm not sure how to fix them as I have
no control on this process.
Cheers,
--Pierre
Ilia Alshanetsky wrote:
> Please revert, its too late for this patch in 5.2.4
>
> Thanks,
>
>
> On 26-Aug-07, at 4:35 PM, Pierre-Alain Joye wrote:
>
>> pajoye Sun Aug 26 20:35:11 2007 UTC
>>
>> Added files: (Branch: PHP_5_2)
>> /php-src/ext/gd/tests libgd00106.phpt
>>
>> Modified files:
>> /php-src/ext/gd/libgd gd.c
>> /php-src NEWS
>> Log:
>> - MFB: gd #106, imagerectangle draws 1x1 rectangles as 1x3 rectangles
>>
>>
>> http://cvs.php.net/viewvc.cgi/php-src/ext/gd/libgd/gd.c?r1=1.90.2.1.2.17&r2=1.90.2.1.2.18&diff_format=u
>> Index: php-src/ext/gd/libgd/gd.c
>> diff -u php-src/ext/gd/libgd/gd.c:1.90.2.1.2.17
php-src/ext/gd/libgd/gd.c:1.90.2.1.2.18
>> --- php-src/ext/gd/libgd/gd.c:1.90.2.1.2.17 Sun Jun 17 19:03:58 2007
>> +++ php-src/ext/gd/libgd/gd.c Sun Aug 26 20:35:10 2007
>> @@ -2124,6 +2124,12 @@
>> int half1 = 1;
>> int t;
>>
>> +
>> + if (x1 == x2 && y1 == y2 && thick == 1) {
>> + gdImageSetPixel(im, x1, y1, color);
>> + return;
>> + }
>> +
>> if (y2 < y1) {
>> t=y1;
>> y1 = y2;
>> http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.910&r2=1.2027.2.547.2.911&diff_format=u
>> Index: php-src/NEWS
>> diff -u php-src/NEWS:1.2027.2.547.2.910 php-src/NEWS:1.2027.2.547.2.911
>> --- php-src/NEWS:1.2027.2.547.2.910 Thu Aug 23 22:39:01 2007
>> +++ php-src/NEWS Sun Aug 26 20:35:11 2007
>> @@ -1,5 +1,6 @@
>> PHP
NEWS
>> |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
>> +- Fixed regression in imagerectangle for 1x1 rectangle (gd bug
#106) (Pierre)