remove unnecessary casts

[stephan-herrmann]

When https://github.com/eclipse-jdt/eclipse.jdt.core/pull/2471 is merged, ecj will signal more casts as unnecessary.

With this PR I suggest to remove affected casts before new errors/warnings will show up in the build.

[github-actions[bot]]

Test Results

   289 files  ±0     289 suites  ±0   58m 39s :stopwatch: - 1m 6s  3 578 tests ±0   3 501 :white_check_mark:  - 1   77 :zzz: +1  0 :x: ±0  10 865 runs  ±0  10 662 :white_check_mark:  - 1  203 :zzz: +1  0 :x: ±0 

Results for commit 7eb2e023. ± Comparison against base commit 34453f99.

This pull request skips 1 test.

``` org.eclipse.pde.build.internal.tests.p2.P2Tests ‑ offBug237662 ```

:recycle: This comment has been updated with latest results.

[stephan-herrmann]

MacOS build is unhappy:

Error:  Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.8:verify (verify) on project org.eclipse.tools.layout.spy: Execute ApiApplication failed: InvocationTargetException: Component 'Layout Spy' in the baseline 'workspace' is disposed: Component was disposed here -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.8:verify (verify) on project org.eclipse.tools.layout.spy: Execute ApiApplication failed

@merks any idea?

[akurtakov]

As long as Jenkins is fine it's good to go.

[stephan-herrmann]

As long as Jenkins is fine it's good to go.

thanks!

[HannesWell]

Thanks for this PR. It's great to have the code, base cleaned-up a bit.

MacOS build is unhappy:

Error:  Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.8:verify (verify) on project org.eclipse.tools.layout.spy: Execute ApiApplication failed: InvocationTargetException: Component 'Layout Spy' in the baseline 'workspace' is disposed: Component was disposed here -

The baseline check is unfortunately flaky sometimes.