Optimize path related utils on windows

[staabm]

the changes mainly prevent excessive use of PCRE on windows. this leads to a small performance improvement of ~3%.

more suprising is blackfire reporting a 56% improvement on memory usage (830 MB instead of 1.9 GB).

tested on windows 11

php -v
PHP 8.1.2 (cli) (built: Jan 19 2022 10:13:52) (NTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.1.2, Copyright (c) Zend Technologies
    with blackfire v1.76.0~win-x64-non_zts81, https://blackfire.io, by Blackfire

[staabm]

moved this normalization below the early return, to prevent unnecessary work

[staabm]

on windows we now take this fast path and prevent PCRE usage for regular paths starting with C:\\

[ondrejmirtes]

What about other drive letters?

[staabm]

are you "ok" with allowing any uppercase character?

[ondrejmirtes]

Yeah, but I feel like there has to be a better way other than comparing letters as if they were numbers :)

[staabm]

wdyt about using ord()? :)

to be honest, I don't like both ways. using regex would work, but we wanne be fast, right? :)

[staabm]

ok, another idea. now using range()

[phpstan-bot]

This pull request has been marked as ready for review.

[ondrejmirtes]

What about other drive letters?

[staabm]

@ondrejmirtes any reasons this was not merged yet? the memory improvement is even more valuable then the time improvement IMO (less likely the process needs to go to swap)

[ondrejmirtes]

I think you're overthinking it - what about not checking the letter at all and just checking that 2nd and 3rd character is :\ ?

[staabm]

I am fine with that, too. here we go.

[ondrejmirtes]

Thank you!