[ 
https://issues.apache.org/jira/browse/LUCENE-4212?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14706390#comment-14706390
 ] 

Uwe Schindler commented on LUCENE-4212:
---------------------------------------

bq. Uwe Schindler I wonder why Math.random() isn't banned by forbiddenApi

I think we should add this. Like with other forbidden-APIs the list may never 
be complete, especially when new Java version come out. E.g. recently I added 
the new Java 8 java.time APIs to forbidden apis (where it used default locale 
or default timezone).

We could add Math.random() to the tests.txt file.

> Tests should not use new Random() without args
> ----------------------------------------------
>
>                 Key: LUCENE-4212
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4212
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: general/build
>            Reporter: Robert Muir
>             Fix For: 4.0-ALPHA, Trunk
>
>         Attachments: LUCENE-4212.patch, LUCENE-4212.patch, LUCENE-4212.patch, 
> LUCENE-4212.patch
>
>
> They should be using random() etc, and if they create one, it should pass in 
> a seed.
> Otherwise, they probably won't reproduce.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to