Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Lucene: Java-Dev

[jira] Commented: (LUCENE-2052) Scan method signatures and add varargs where possible

 

 

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded


jira at apache

Nov 10, 2009, 3:58 PM

Post #1 of 4 (248 views)
Permalink
[jira] Commented: (LUCENE-2052) Scan method signatures and add varargs where possible

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

Uwe Schindler commented on LUCENE-2052:
---------------------------------------

Is there any eclipse plugin/refactoring/cleanup that helps when doing this? Cannot find anything :-(

> Scan method signatures and add varargs where possible
> -----------------------------------------------------
>
> Key: LUCENE-2052
> URL: https://issues.apache.org/jira/browse/LUCENE-2052
> Project: Lucene - Java
> Issue Type: Improvement
> Affects Versions: 3.0
> Reporter: Uwe Schindler
> Assignee: Uwe Schindler
> Fix For: 3.0
>
>
> I changed a lot of signatures, but there may be more. The important ones like MultiReader and MultiSearcher are already done. This applies also to contrib. Varargs are no backwards break, they stay arrays as before.

--
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: java-dev-unsubscribe [at] lucene
For additional commands, e-mail: java-dev-help [at] lucene


jira at apache

Nov 14, 2009, 12:04 AM

Post #2 of 4 (218 views)
Permalink
[jira] Commented: (LUCENE-2052) Scan method signatures and add varargs where possible [In reply to]

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

Uwe Schindler commented on LUCENE-2052:
---------------------------------------

Thanks, I didn't check contrib.

> Scan method signatures and add varargs where possible
> -----------------------------------------------------
>
> Key: LUCENE-2052
> URL: https://issues.apache.org/jira/browse/LUCENE-2052
> Project: Lucene - Java
> Issue Type: Improvement
> Affects Versions: 3.0
> Reporter: Uwe Schindler
> Assignee: Uwe Schindler
> Fix For: 3.0
>
> Attachments: LUCENE-2052.patch, LUCENE-2052.patch, LUCENE-2052_fa.patch
>
>
> I changed a lot of signatures, but there may be more. The important ones like MultiReader and MultiSearcher are already done. This applies also to contrib. Varargs are no backwards break, they stay arrays as before.

--
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: java-dev-unsubscribe [at] lucene
For additional commands, e-mail: java-dev-help [at] lucene


jira at apache

Nov 14, 2009, 1:18 AM

Post #3 of 4 (217 views)
Permalink
[jira] Commented: (LUCENE-2052) Scan method signatures and add varargs where possible [In reply to]

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

Robert Muir commented on LUCENE-2052:
-------------------------------------

bq. Thanks, I didn't check contrib.

I didn't really completely check, not sure really where we should have varargs there.
(it does not make sense for stopwords to me, but then I feel all we need is Set<?> instead of file, varargs, hashtable, and all the other constructors we have)

still i'd rather have consistency for now.

> Scan method signatures and add varargs where possible
> -----------------------------------------------------
>
> Key: LUCENE-2052
> URL: https://issues.apache.org/jira/browse/LUCENE-2052
> Project: Lucene - Java
> Issue Type: Improvement
> Affects Versions: 3.0
> Reporter: Uwe Schindler
> Assignee: Uwe Schindler
> Fix For: 3.0
>
> Attachments: LUCENE-2052.patch, LUCENE-2052.patch, LUCENE-2052_fa.patch
>
>
> I changed a lot of signatures, but there may be more. The important ones like MultiReader and MultiSearcher are already done. This applies also to contrib. Varargs are no backwards break, they stay arrays as before.

--
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: java-dev-unsubscribe [at] lucene
For additional commands, e-mail: java-dev-help [at] lucene


jira at apache

Nov 14, 2009, 1:31 AM

Post #4 of 4 (218 views)
Permalink
[jira] Commented: (LUCENE-2052) Scan method signatures and add varargs where possible [In reply to]

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

Uwe Schindler commented on LUCENE-2052:
---------------------------------------

Yes, that's right. For stopwords a varargs ctor is not really "the right thing". Varargs are more for cases like instantiating a MultiReader with 5 IndexReaders or something like that. Or adding a bunch of clauses to a SpanQuery.
The varargs Set<?> is one unit, arrays are not needed (they are there because of BW compatibility). All these ctors except Set<?> should have been deprecated in 2.9.

> Scan method signatures and add varargs where possible
> -----------------------------------------------------
>
> Key: LUCENE-2052
> URL: https://issues.apache.org/jira/browse/LUCENE-2052
> Project: Lucene - Java
> Issue Type: Improvement
> Affects Versions: 3.0
> Reporter: Uwe Schindler
> Assignee: Uwe Schindler
> Fix For: 3.0
>
> Attachments: LUCENE-2052.patch, LUCENE-2052.patch, LUCENE-2052_fa.patch
>
>
> I changed a lot of signatures, but there may be more. The important ones like MultiReader and MultiSearcher are already done. This applies also to contrib. Varargs are no backwards break, they stay arrays as before.

--
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: java-dev-unsubscribe [at] lucene
For additional commands, e-mail: java-dev-help [at] lucene

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.