[wp-trac] [WordPress Trac] #58206: Replace usage of strpos with str_contains

[WordPress Trac]

#58206: Replace usage of strpos with str_contains
-------------------------------------+-------------------------------------
 Reporter:  spacedmonkey             |       Owner:  SergeyBiryukov
     Type:  enhancement              |      Status:  assigned
 Priority:  normal                   |   Milestone:  6.3
Component:  General                  |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  good-first-bug has-      |     Focuses:  performance, coding-
  patch 2nd-opinion commit           |  standards
-------------------------------------+-------------------------------------

Comment (by azaozz):

 Replying to [comment:13 spacedmonkey]:
 > I run benchmarks against this change. I see a net posative.

 Hmmmm, that's really strange? Why would the benchmark you're using miss
 the 300% slow-down of the PHP < 8 native code vs. the polyfill, and the
 PHP 8+ native code? Seems something's not right?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/58206#comment:16>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform