[DeadCode] Remove property comment same line on RemoveUnusedPrivatePropertyRector

[samsonasik]

Closes #3539

[samsonasik]

@staabm this is a quick solution, the general solution probably on the removeNode() process, but that will make it require next node attribute usage

[samsonasik]

All checks have passed 🎉 @TomasVotruba it is ready for review.

[samsonasik]

@staabm another general solution is probably create a dedicated rector rule to verify the Stmt just removed with next is Nop in the same line, eg: RemoveJustRemovedSameLineCommentStmtRemovedRector

that will work on usage on other rule as well :)

[staabm]

Thanks for working on it.

I can't tell which works best for rector. I have no experience with Nop AST nodes either.

[samsonasik]

I will create alternative PR for it 👍

[samsonasik]

Checking stmt just removed in loop of StmtsAwareInterface probably make memory hog as it will compare object in array, so probably not best solution.

[samsonasik]

@staabm I created alternative PR for it, nevertheles #3548