<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2014-09-23 20:04 GMT+02:00 Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hugues Peccatte a écrit :<br>
<div><div class="h5">> 2014-09-23 16:23 GMT+02:00 Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>:<br>
><br>
> That's it. Instead of using $pmaString->strlen or strlen, we could imagine<br>
> to use pmaStrlen and this function would call strlen or mb_strlen if mb_*<br>
> functions exist.<br>
> Another way could be to overwrite strlen with mb_strlen, but we wouldn't be<br>
> able to use the original strlen if needed. So I think that we should define<br>
> new functions.<br>
<br>
</div></div>What I don't like is that we would have two different syntaxes to call<br>
string functions. We would need comments in the calling code to explain<br>
why we are using a way and not another way.<br>
<span class="im HOEnZb"><br>
><br>
> I'll try to do a mass replacement and check if the execution time is better.<br>
><br>
> Hugues.<br>
<br>
</span><div class="HOEnZb"><div class="h5">--<br>
Marc Delisle (phpMyAdmin)</div></div></blockquote></div><br></div><div class="gmail_extra">If by "two ways", you think about PMA_String and pmaStr* functions, there should be a misunderstanding. I don't want to keep PMA_String (at least, not for string functions), I'd like to migrate it into functions. Here, I kept it not to break anything, just to check performance.</div><div class="gmail_extra">If you think about pmaStr* functions and standard PHP functions, I don't think this is a real issue, because we use PMA string object almost everywhere today, so it would be replaced by function calls.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Hugues.</div></div>