Uploaded image for project: 'Causeway'
  1. Causeway
  2. CAUSEWAY-2117

Don't use shared instances of 'Random'

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • None
    • 2.0.0-M3
    • None
    • None
    • Patch

    Description

      Replace Math.random with Random.nextDouble. There is a performance overhead cost when using Math.random over Random.nextDouble. Even moreso, the Random class can allow for easier random manipulation in the future if needed.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            hobrom Andi Huber
            bd2019us bd2019us
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 20m
                20m

                Slack

                  Issue deployment