chore: more java17 idiomatic code

[robfrank]

What does this PR do?

Chore PR: more java17 compliant code

Motivation

to java 17 and beyond

Checklist

• I have run the build using mvn clean package command
• My unit tests cover both failure and success scenarios

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
✅ +0.02%	✅ 71.83%

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (ba1ba1d)	68823	44204	64.23%
Head commit (3f049da)	68795 (-28)	44203 (-1)	64.25% (+0.02%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#1806)	323	232	71.83%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

_{Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more}

[lvca]

I see many changes are just formatting, like:

 if (ov != null)
          neighbors.add(ov);

into

 if (ov != null) neighbors.add(ov);

I think it's not Java17, that your formatting setting :-)