Index: lucene/core/src/java/org/apache/lucene/analysis/CharReader.java =================================================================== --- lucene/core/src/java/org/apache/lucene/analysis/CharReader.java (revision 1326494) +++ lucene/core/src/java/org/apache/lucene/analysis/CharReader.java (working copy) @@ -55,6 +55,11 @@ } @Override + public int read() throws IOException { + return input.read(); + } + + @Override public boolean markSupported(){ return input.markSupported(); } Index: modules/analysis/common/src/java/org/apache/lucene/analysis/charfilter/CharFilter.java =================================================================== --- modules/analysis/common/src/java/org/apache/lucene/analysis/charfilter/CharFilter.java (revision 1326494) +++ modules/analysis/common/src/java/org/apache/lucene/analysis/charfilter/CharFilter.java (working copy) @@ -66,6 +66,11 @@ } @Override + public int read() throws IOException { + return input.read(); + } + + @Override public boolean markSupported(){ return input.markSupported(); } Index: modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java =================================================================== --- modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java (revision 1326494) +++ modules/analysis/common/src/test/org/apache/lucene/analysis/core/TestRandomChains.java (working copy) @@ -756,77 +756,32 @@ } } - // wants charfilter to be a filterreader... - // do *NOT* refactor me to be a charfilter: LUCENE-3990 - static class CheckThatYouDidntReadAnythingReaderWrapper extends CharStream { - boolean readSomething; - CharStream in; + static final class CheckThatYouDidntReadAnythingReaderWrapper extends CharFilter { + boolean readSomething = false; CheckThatYouDidntReadAnythingReaderWrapper(Reader in) { - this.in = CharReader.get(in); + super(CharReader.get(in)); } - - @Override - public int correctOffset(int currentOff) { - return in.correctOffset(currentOff); - } @Override - public void close() throws IOException { - in.close(); - } - - @Override public int read(char[] cbuf, int off, int len) throws IOException { readSomething = true; - return in.read(cbuf, off, len); + return super.read(cbuf, off, len); } @Override public int read() throws IOException { readSomething = true; - return in.read(); + return super.read(); } @Override - public int read(CharBuffer target) throws IOException { - readSomething = true; - return in.read(target); - } - - @Override - public void mark(int readAheadLimit) throws IOException { - in.mark(readAheadLimit); - } - - @Override - public boolean markSupported() { - return in.markSupported(); - } - - @Override - public int read(char[] cbuf) throws IOException { - readSomething = true; - return in.read(cbuf); - } - - @Override - public boolean ready() throws IOException { - return in.ready(); - } - - @Override - public void reset() throws IOException { - in.reset(); - } - - @Override public long skip(long n) throws IOException { readSomething = true; - return in.skip(n); + return super.skip(n); } } - + static class TokenizerSpec { Tokenizer tokenizer; String toString;