Re: [PATCH] Bug #46806 - mb_strimwidth

Hi, This behavior seems strange, but the rationale of this function is that the (east asian) width of the resulting string does not exceed the specified value so that it would fit to a fixed sized box when rendered in the browser, assuming the string would be displayed with a monospace font. So, it is the document that is wrong. Regards, Moriyoshi (2008/12/31 1:24), Henrique M. Decaria wrote:

Hello guys, Looking in the subject mentioned bug report, it seems that the attached patch might do the trick and make the mb_strimwidth() function work as explained in the php manual (http://br2.php.net/mb_strimwidth). According to it, the function should trim up to the defined width and append the trimmarker, that being said, the following code should return "hello..." not "he..." echo mb_strimwidth('helloworld', 0, 5, '...', 'UTF-8'); The change in the file is:

-    pc.width = width - mkwidth;
+    pc.width = width;

I am not sure why there is this "- mkwidth" there, but it seems to be used to remove the last "mkwidth" characters from the trimmed string so it would be replaced with the trimmarker. It could be that the test file for this function is incorrect (i can't guarantee as I am having issues with Japanese characters here). So please, if you could have a look and see if it works as expected with this patch it would be great. Thanks, Henrique

-- -- moriyoshi