lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Muir (JIRA)" <>
Subject [jira] Commented: (LUCENE-2094) Prepare CharArraySet for Unicode 4.0
Date Sun, 29 Nov 2009 12:46:21 GMT


Robert Muir commented on LUCENE-2094:

Hi Simon,

One thing I noticed is with this patch we get:
public StopFilter(Version matchVersion, boolean enablePositionIncrements, TokenStream input,
Set<?> stopWords, boolean ignoreCase)

I know this is really not related to what you are doing here, but I wonder if instead StopFilter
should look like this:
public StopFilter(Version matchVersion, TokenStream input, Set<?> stopWords, boolean

and use matchVersion to determine enablePositionIncrements. 

I think its already wierd how to create a stopfilter, you have to pass version to a static
method getEnablePositionIncrementsVersionDefault. I don't think the user should have to pass
Version twice:
new StopFilter(Version.WHATEVER, StopFilter.getEnablePositionIncrementsVersionDefault(Version.WHATEVER),

I guess i think this getEnablePositionIncrementsVersionDefault should be deprecated along
with the ctors that take this boolean argument, and it should all be driven off a single Version
argument for simplicity

> Prepare CharArraySet for Unicode 4.0
> ------------------------------------
>                 Key: LUCENE-2094
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Analysis
>    Affects Versions: 1.9, 2.0.0, 2.1, 2.2, 2.3, 2.3.1, 2.3.2, 2.3.3, 2.4, 2.4.1, 2.4.2,
2.9, 2.9.1, 2.9.2, 3.0, 3.0.1, 3.1
>            Reporter: Simon Willnauer
>             Fix For: 3.1
>         Attachments: LUCENE-2094.patch, LUCENE-2094.txt, LUCENE-2094.txt, LUCENE-2094.txt
> CharArraySet does lowercaseing if created with the correspondent flag. This causes that
 String / char[] with uncode 4 chars which are in the set can not be retrieved in "ignorecase"

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message