Index: lucene/analysis/common/src/java/org/apache/lucene/analysis/ngram/EdgeNGramTokenizerFactory.java =================================================================== --- lucene/analysis/common/src/java/org/apache/lucene/analysis/ngram/EdgeNGramTokenizerFactory.java (revision 1463223) +++ lucene/analysis/common/src/java/org/apache/lucene/analysis/ngram/EdgeNGramTokenizerFactory.java (working copy) @@ -40,8 +40,8 @@ /** Creates a new EdgeNGramTokenizerFactory */ public EdgeNGramTokenizerFactory(Map args) { super(args); - minGramSize = getInt(args, "minGramSize", EdgeNGramTokenFilter.DEFAULT_MIN_GRAM_SIZE); - maxGramSize = getInt(args, "maxGramSize", EdgeNGramTokenFilter.DEFAULT_MAX_GRAM_SIZE); + minGramSize = getInt(args, "minGramSize", EdgeNGramTokenizer.DEFAULT_MIN_GRAM_SIZE); + maxGramSize = getInt(args, "maxGramSize", EdgeNGramTokenizer.DEFAULT_MAX_GRAM_SIZE); String sideArg = args.remove("side"); if (sideArg == null) { Index: lucene/analysis/common/src/java/org/apache/lucene/analysis/pattern/PatternTokenizerFactory.java =================================================================== --- lucene/analysis/common/src/java/org/apache/lucene/analysis/pattern/PatternTokenizerFactory.java (revision 1463223) +++ lucene/analysis/common/src/java/org/apache/lucene/analysis/pattern/PatternTokenizerFactory.java (working copy) @@ -71,13 +71,8 @@ public PatternTokenizerFactory(Map args) { super(args); pattern = getPattern(args, PATTERN); - - String v = args.remove(GROUP); - if (v == null) { - group = -1; // use 'split' - } else { - group = Integer.parseInt(v); - } + group = getInt(args, GROUP, -1); + if (!args.isEmpty()) { throw new IllegalArgumentException("Unknown parameters: " + args); } Index: lucene/analysis/common/src/test/org/apache/lucene/analysis/charfilter/TestMappingCharFilterFactory.java =================================================================== --- lucene/analysis/common/src/test/org/apache/lucene/analysis/charfilter/TestMappingCharFilterFactory.java (revision 1463223) +++ lucene/analysis/common/src/test/org/apache/lucene/analysis/charfilter/TestMappingCharFilterFactory.java (working copy) @@ -17,14 +17,12 @@ * limitations under the License. */ -import java.util.HashMap; - import org.apache.lucene.analysis.util.BaseTokenStreamFactoryTestCase; public class TestMappingCharFilterFactory extends BaseTokenStreamFactoryTestCase { public void testParseString() throws Exception { - MappingCharFilterFactory f = new MappingCharFilterFactory(new HashMap()); + MappingCharFilterFactory f = (MappingCharFilterFactory)charFilterFactory("Mapping"); try { f.parseString( "\\" );