[BEAM-408] ProxyInvocationHandler uses inefficient Math.random() for random int

• Type: Bug
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: 0.2.0-incubating
• Component/s: sdk-java-core
• Labels: Done findbugs newbie starter
• Assignee: Lucas Amorim
• Reporter: Scott Wegner
• Created: 30/Jun/16 22:27
• Updated: 06/Oct/20 21:50
• Resolved: 07/Jul/16 13:47

Description

FindBugs DM_NEXTINT_VIA_NEXTDOUBLE: Use the nextInt method of Random rather than nextDouble to generate a random integer

Applies to: ProxyInvocationHandler.hashCode. This random value is used as a pseudo-unique value for #hashCode(), although the default Object.hasCode() implementation does the same thing, so this should be unnecessary.

This is a good starter bug. When fixing, please remove the corresponding entries from findbugs-filter.xml and verify the build passes.

Issue Links

• links to: GitHub Pull Request #588