[Performance][NodeTypeResolver] Remove $mutatingScope->enterCatchType() usage on PHPStanNodeScopeResolver

[samsonasik]

@staabm if I remember correctly, $mutatingScope->enterCatchType() on try catch was used on detecting undefined variable which the rule is removed at PR:

• [DX] Remove AddDefaultValueForUndefinedVariableRector as can generate broken code and depends on context #4729

I think it save to be removed to reduce memory usage. I will check if this compatible with different projects after this merged.

[staabm]

looks cool (as long as build keeps green)

[samsonasik]

All checks have passed 🎉 @TomasVotruba I am merging it to have faster feedback to test :)

[samsonasik]

@staabm it seems it cause error on downgrade:

                                                                                
 [ERROR] Could not process                                                      
         "/home/runner/work/rector-src/rector-src/rector-build/vendor/react/even
         t-loop/src/StreamSelectLoop.php" file, due to:                         
         "System error: "Scope not available on "PhpParser\Node\Stmt\Expression"
         node, but is required by a refactorWithScope() method of               
         "Rector\DowngradePhp80\Rector\Expression\DowngradeMatchToSwitchRector" 
         rule. Fix scope refresh on changed nodes first"                        
         Run Rector with "--debug" option and post the report here: 
/~https://github.com/rectorphp/rector/issues/new
". On line:
         45                                                                     
                                                                                

I will revert.

[samsonasik]

Reverting at #5046

[staabm]

we should really have this downgrade job here on the repo :-/