Go for merge when ready 👍

@TomasVotruba I am thinking that it may be slow down the process refactor as it require another loop after node refactored for usage of new visitor to loop node stmts just to clearance the Nop usage.

It also rely on NodesToRemoveCollector usage, which it will still buggy on remove by $node->stmts[$key] usage in other rule if used as no previous key may be 2 things;

• previous stmt never exists before.
• previous stmt just unsetted.

I think we can have this case by case for it, so closing it for now.

I see, makes sense 👍