[wp-trac] [WordPress Trac] #58206: Replace usage of strpos with str_contains
WordPress Trac
noreply at wordpress.org
Mon May 29 19:01:03 UTC 2023
#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 spacedmonkey):
Maybe the performance lead, @flixos90 could take a look into this.
To be clear, if they’re a slight performance loss, I think this change is
worth for the code quality improvement.
Worth noting, the performance team is currently working on docs on how we
run our benchmarks. That way we could run the same benchmarks and we could
see similar results.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/58206#comment:17>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list