Re: com php-src: imagettftext() uses the last charactermap it found and not the one you want: NEWS ext/gd/libgd/gd.h ext/gd/libgd/gdft.c
ext/gd/tests/Rochester-Regular.otf ext/gd/tests/Rochester-Regular.otf.LICENSE.txt ext/gd/tests/imagettftext_charmap_order.phpt

From: Date: Mon, 17 Feb 2014 07:59:21 +0000
Subject: Re: com php-src: imagettftext() uses the last charactermap it found and not the one you want: NEWS ext/gd/libgd/gd.h ext/gd/libgd/gdft.c
ext/gd/tests/Rochester-Regular.otf ext/gd/tests/Rochester-Regular.otf.LICENSE.txt ext/gd/tests/imagettftext_charmap_order.phpt
References: 1  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-76034@lists.php.net to get a copy of this message
Hi Scott, Good enough for now, thanks :) Did you forget to test the MacRoman in fontTest? It would be great if you coould provide a PR in bitbuckket's repo too, which hhvm would most likely follow too :) Cheers, On Mon, Feb 17, 2014 at 7:49 AM, Scott MacVicar <scottmac@php.net> wrote: > Commit: 796bfb0bd000f7d9d23e633a965f6a8a77a5e927 > Author: Scott MacVicar <scottmac@php.net> Sun, 16 Feb 2014 21:53:19 -0800 > Parents: 1f3af9cd2152f08bb52bb69d035a4aefdd9d13bf > Branches: master > > Link: > http://git.php.net/?p=php-src.git;a=commitdiff;h=796bfb0bd000f7d9d23e633a965f6a8a77a5e927 > > Log: > imagettftext() uses the last charactermap it found and not the one you want > > Summary: > This caused issues when you picked a font with multiple maps, it wouldn't > pick the unicode version but instead pick the last one it saw. Change so > we cache on a per encoding level. There was even a todo list item in the code. > > In an ideal world we can probably iterate through at font write time. > > Test Plan: > See said unit test, it's Apache Licensed font included > > Changed paths: > M NEWS > M ext/gd/libgd/gd.h > M ext/gd/libgd/gdft.c > A ext/gd/tests/Rochester-Regular.otf > A ext/gd/tests/Rochester-Regular.otf.LICENSE.txt > A ext/gd/tests/imagettftext_charmap_order.phpt > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php -- Pierre @pierrejoye | http://www.libgd.org

« previous php.cvs (#76034) next »