Opened 15 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.