Erick / Mike -
With 2065 commited onto trunk now - I created another patch for 2037
and attached in the ticket.
3 classes remain pending though due to conflicts , that I had listed
with the patch. But we can probably revisit them subsequently. Please
review them to serve as a starting point for the same.
Erick Erickson wrote:
Mike:
I should be able to create a new 2037 patch pretty easily if you
want to apply 2065 first. Let me know....
Erick
On Thu, Dec 3, 2009 at 9:05 PM, Kay Kay <kaykay.uni...@gmail.com
<mailto:kaykay.uni...@gmail.com>> wrote:
Mike -
I have attached another patch to LUCENE-2065 , in sync with the
trunk now.
Erick Erickson wrote:
That's up to Mike, whichever way he finds easiest, I'll deal.
Erick
On Thu, Dec 3, 2009 at 8:43 PM, Kay Kay
<kaykay.uni...@gmail.com <mailto:kaykay.uni...@gmail.com>
<mailto:kaykay.uni...@gmail.com
<mailto:kaykay.uni...@gmail.com>>> wrote:
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>
<mailto:luc...@mikemccandless.com
<mailto:luc...@mikemccandless.com>>
<mailto:luc...@mikemccandless.com
<mailto:luc...@mikemccandless.com>
<mailto: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>
<mailto:erickerick...@gmail.com <mailto:erickerick...@gmail.com>>
<mailto:erickerick...@gmail.com
<mailto:erickerick...@gmail.com>
<mailto: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>
<mailto:java-dev-unsubscr...@lucene.apache.org
<mailto:java-dev-unsubscr...@lucene.apache.org>>
<mailto:java-dev-unsubscr...@lucene.apache.org
<mailto:java-dev-unsubscr...@lucene.apache.org>
<mailto: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>
<mailto:java-dev-h...@lucene.apache.org
<mailto:java-dev-h...@lucene.apache.org>>
<mailto:java-dev-h...@lucene.apache.org
<mailto:java-dev-h...@lucene.apache.org>
<mailto:java-dev-h...@lucene.apache.org
<mailto:java-dev-h...@lucene.apache.org>>>
---------------------------------------------------------------------
To unsubscribe, e-mail:
java-dev-unsubscr...@lucene.apache.org
<mailto:java-dev-unsubscr...@lucene.apache.org>
<mailto: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>
<mailto:java-dev-h...@lucene.apache.org
<mailto:java-dev-h...@lucene.apache.org>>
---------------------------------------------------------------------
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