BC: Separated regex matching in NodeNameResolver->isName()

[staabm]

as discussed in #4950 (comment)

---

Afte todays patches rectorphp is running 100 seconds faster over the Mautic codebase.

latest stable release takes ~ 625.33s
rector@dev takes ~ 523s on my machine

.. and requires way less RAM

[samsonasik]

@staabm please rebase to fix conflict, thank you.

[TomasVotruba]

@staabm rector-symfony needs one last check, that name is not Expr. Then its ready to go 👍

[TomasVotruba]

Thanks 👍

@staabm How are we doing on the performance numbers? :)

[staabm]

updated the PRs description above

[TomasVotruba]

That's awesome improvement 👏 👏 👏

I wonder if we use regex by default somewhere else but actually don't need it 🤔

[samsonasik]

@staabm it seems cause error on rector-phpunit:

• [CodeQuality] Skip call parent::__construct() indirect parent on ConstructClassMethodToSetUpTestCaseRector rector-phpunit#251