PHP Fixes found by Static Analysis Tool#1261
PHP Fixes found by Static Analysis Tool#1261craiglondon wants to merge 8 commits intophpseclib:masterfrom
Conversation
|
I like how it added the Thanks! |
|
I think the only commit related to Amy performance (supposed) improvements is this one. I haven't done any benchmarks before but would be willing to do some given a little direction as a best tool to use? like using Blackfire and PHPunit? |
|
I've never used Blackfire lol. The way I normally benchmark stuff is just to loop over it a million times lol. And in this case I'm not sure a loop would be the most effective way to do the benchmark since I don't really expect anyone to do that stuff in a loop. I guess two possible ideas for benchmarking:
|
|
Why test performance of comparison operators? It's known that strict comparison is faster and benchmark proves it (approx 15% faster) https://github.com/8ctopus/php-benchmark Typehint performance difference is negligible 0.4 ms per 100M cycles |
No description provided.