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

Yonik Seeley commented on LUCENE-3410:
--------------------------------------

For historical context, the reason I used an int for stuff like 
generateWordParts was that I had the idea of using it as a minimum (i.e. only 
generate word parts that are over a certain size, etc).  This obviously never 
happened though ;-)

> Make WordDelimiterFilter's instantiation more readable
> ------------------------------------------------------
>
>                 Key: LUCENE-3410
>                 URL: https://issues.apache.org/jira/browse/LUCENE-3410
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: modules/analysis
>            Reporter: Chris Male
>            Priority: Minor
>
> Currently WordDelimiterFilter's constructor is:
> {code}
> public WordDelimiterFilter(TokenStream in,
>                                    byte[] charTypeTable,
>                                    int generateWordParts,
>                                    int generateNumberParts,
>                                    int catenateWords,
>                                    int catenateNumbers,
>                                    int catenateAll,
>                                    int splitOnCaseChange,
>                                    int preserveOriginal,
>                                    int splitOnNumerics,
>                                    int stemEnglishPossessive,
>                                    CharArraySet protWords) {
> {code}
> which means its instantiation is an unreadable combination of 1s and 0s.  
> We should improve this by either using a Builder, 'int flags' or an EnumSet.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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

Reply via email to