XenoAmess
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...