search

AdoptOpenJDK / jitwatch

Log analyser / visualiser for Java HotSpot JIT compiler. Inspect inlining decisions, hot methods, bytecode, and assembly. View results in the JavaFX user interface.
Other
3.06k stars 437 forks source link

Don't use deprecated methods #320

Closed JCWasmx86 closed 4 years ago

JCWasmx86 commented 4 years ago

The constructor new Boolean(...) is deprecated and newInstance(), too.

chriswhocodes commented 4 years ago

Thanks for highlighting this. Looking at the code I can't see why it would even need the Boolean wrapper at all. Possibly left behind from a refactoring. Would you mind cleaning this up in your patch? Thanks!

JCWasmx86 commented 4 years ago

Ok, I cleaned up JavapTask, Should I cleanup the generics too, as they often don't use the diamond operator, although it is available since Java 7?

chriswhocodes commented 4 years ago

Thank you for these clean ups!