Opened 14 years ago
Closed 14 years ago
#17754 closed enhancement (fixed)
preg_match() use is overkill in path_is_absolute()
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.3 | Priority: | normal |
Severity: | minor | Version: | 3.1.4 |
Component: | Performance | Keywords: | has-patch |
Focuses: | Cc: |
Description
path_is_absolute()
utilizes preg_match()
to check if the first character of a path string is either of two possibilities. This is way overkill for such a simple case. The alternative I've proposed via the attached patch is more performant and makes the intention of the code clearer (rather than obscured in a regular expression).
Attachments (2)
Change History (6)
Note: See
TracTickets for help on using
tickets.
Like the patch, but our coding standards (or at least standard usage) are for array style access to characters i.e.
$str[0]
. See #13900.