XenoAmess

Results 184 comments of XenoAmess

@garydgregory rebased. please find some time to review. thanks.

@arturobernalg Got it. I got a really busy job recently, sorry for the late reply.

@garydgregory rebased. please find some time to review. thanks.

> Hm, why do we want something so much more complicated? @garydgregory mostly for performance reason

> Would you, please, be so kind to rebase again? Sorry for that inconvenience. @jochenw rebase done.

@aherbert fixed. please re-review. thanks.

Why not change to use `java.security.SecureRandom` here? Is there be any reason for not using it instead? What about adding a configuration boolean param about it?

If there should have an IllegalArgumentExceptions , then there should also have a RuntimeExceptions. then there should also have ArithmeticExceptions. and should also have IndexOutOfBoundsExceptions, FileSystemNotFoundExceptions ...... I don't think...

feel free to ask for changes about this test, I will help to run it. Then we decide whether change usage of WeakHashMap to CuncurrentHashMap, or remain what it looks...

> Looks like you have some merge conflicts? Sorry for that. will fix it immediately. > Also I think this will resolve BEANUTILS-509: #21 I had no experience in using...