Removing is_array calls in PropertyHolder
Removing is_array calls in PropertyHolder

This is a micro optimization, it’s not significant when running just
one agent through the parser, but when running thousands of them, it
does save a bit of time as seen in these two profiler runs (few hundred
ms):

 * with `in_array`:
https://blackfire.io/profiles/9351d1db-13f7-4178-8cde-f250829a915a/graph
?settings%5Bdimension%5D=wt&settings%5Bdisplay%5D=landscape&settings%5Bt
abPane%5D=nodes&selected=&callname=main()
 * without `in_array` (this commit):
https://blackfire.io/profiles/219f75dd-e818-4d85-94ec-169ddb9a3ef6/graph
?settings%5Bdimension%5D=wt&settings%5Bdisplay%5D=landscape&settings%5Bt
abPane%5D=nodes&selected=&callname=main()

(in_array is called 235,708 times, at a cost of ~200ms)

These profile runs were against a list of 1,000 useragents.

mimmi20/browscap-php,mimmi20/browscap-php,browscap/browscap-php