You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@lucene.apache.org by rm...@apache.org on 2012/04/09 15:44:19 UTC

svn commit: r1311235 - /lucene/dev/branches/lucene3969/modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java

Author: rmuir
Date: Mon Apr  9 13:44:18 2012
New Revision: 1311235

URL: http://svn.apache.org/viewvc?rev=1311235&view=rev
Log:
LUCENE-3969: don't be this evil yet for type char

Modified:
    lucene/dev/branches/lucene3969/modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java

Modified: lucene/dev/branches/lucene3969/modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene3969/modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java?rev=1311235&r1=1311234&r2=1311235&view=diff
==============================================================================
--- lucene/dev/branches/lucene3969/modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java (original)
+++ lucene/dev/branches/lucene3969/modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java Mon Apr  9 13:44:18 2012
@@ -188,7 +188,14 @@ public class TestRandomChains extends Ba
     });
     put(char.class, new ArgProducer() {
       @Override public Object create(Random random) {
-        return Character.valueOf((char)random.nextInt(65536));
+        // nocommit: fix any filters that care to throw IAE instead.
+        // return Character.valueOf((char)random.nextInt(65536));
+        while(true) {
+          char c = (char)random.nextInt(65536);
+          if (c < '\uD800' || c > '\uDFFF') {
+            return Character.valueOf(c);
+          }
+        }
       }
     });
     put(float.class, new ArgProducer() {