Refactor ObjectComparison to utility class

[SougandhS]

What it does

Convert ObjectComparison to a static utility class by hiding the constructor and making all methods static.

How to test

Author checklist

• I have thoroughly tested my changes
• The change is following the coding conventions
• I have signed the Eclipse Contributor Agreement (ECA)

[iloveeclipse]

Why a singleton is needed at all? The object seems to be stateless, so actually it is a bunch of static helper methods?

[SougandhS]

I noticed multiple instances being created during repeated operations in visualVM so tried a singleton to avoid that. However, as you pointed the class is stateless, I believe this optimization isn’t really necessary. Will close this PR then

[iloveeclipse]

maybe it would be better to

1. hide public constructor
2. make all methods static
3. "import static *" in the places where this code is used

This will prevent later confusion if someone would start adding state to this utility class...

[SougandhS]

maybe it would be better to

1. hide public constructor
2. make all methods static
3. "import static *" in the places where this code is used

This will prevent later confusion if someone would start adding state to this utility class...

Sure 👍

[SougandhS]

Will merge this soon 👍

[SougandhS]

I hope its good to be merged now