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> 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>> 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 > >