I created Lucene-2065 while working on 1257 , the original generics
related ticket , and since we were running out of time for 3.0 , I
guess we could not get src/test converted in.
In any case , if you were comitting this one (2037) to trunk , may be I
can wait before creating the patch again.
Erick Erickson wrote:
I didn't realize 2065 had already been down this path, thought
you were volunteering to change all the code starting from
scratch. Your approach sounds like a fine plan.
Note that I'm not entirely sure that I cleaned up *everything*, but we
need to get to a known state before tackling the rest, so I'll wait for
these two patches to be applied before looking back at it...
Not to mention the Localized test thing.....
Erick
On Thu, Dec 3, 2009 at 5:57 PM, Michael McCandless
<luc...@mikemccandless.com <mailto:luc...@mikemccandless.com>> wrote:
On Thu, Dec 3, 2009 at 5:48 PM, Erick Erickson
<erickerick...@gmail.com <mailto:erickerick...@gmail.com>> wrote:
> I generified the searches/function files in patch 2037. I don't
really think
> there's a conflict, just commit my patch and have at generifying
the rest.
OK so then we'll start with 2037, then take 2065's patch, hopefully
updated to current trunk, but minus search/function sources.
> I know, I know. I did two things at once. So sue me. Honest,
I'll try not to
> do this very often <G>...
In fact I prefer this. I used to think we shouldn't do that but I
flip-flopped and now think in practice you just have to clean code
while you're there, otherwise it won't get cleaned.
> Mike:
> You really want to to the generify the whole shootin' match or
do you want
> to partition them? I'll be happy to take a set of them. Or would
that make
> things too complicated to apply?
2065 already has done alot here (adding generics to the tests)... I
think we start from that and take it from there?
Mike
---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org
<mailto:java-dev-unsubscr...@lucene.apache.org>
For additional commands, e-mail: java-dev-h...@lucene.apache.org
<mailto:java-dev-h...@lucene.apache.org>
---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: java-dev-h...@lucene.apache.org
