farhanrahman / kyoto

18 stars 7 forks source link

Class kyoto/services/Economy#updateEconomyState uses a new Random in each invocation #120

Closed gubby closed 12 years ago

gubby commented 12 years ago

A new Random instance is seeded based on the System.nanoTime which means the sequence will be predictable.

piotroslaw commented 12 years ago

This seems like the last thing we should be worrying about right now to be honest.

If I misunderstood and this is really a game-breaking problem please reopen this, but closing for now

gubby commented 12 years ago

Its a broken feature and a 2 second fix. Its your project, I'm not re-opening anything! You're welcome anyway.

On 22 June 2012 15:01, Piotr Dittrich < reply@reply.github.com

wrote:

This seems like the last thing we should be worrying about right now to be honest.

If I misunderstood and this is really a game-breaking problem please reopen this, but closing for now


Reply to this email directly or view it on GitHub: https://github.com/farhanrahman/kyoto/issues/120#issuecomment-6508289