OK I'll apply 2065 first, then we'll iterate on 2037 and get it in -- thanks Erick and Kay Kay! It's wonderful you all are putting in the energy to keep our tests current :)
Mike On Thu, Dec 3, 2009 at 9:14 PM, Erick Erickson <erickerick...@gmail.com> 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> 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>> 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>>> 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>>> 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>> >>> >>> 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 >> > > --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org For additional commands, e-mail: java-dev-h...@lucene.apache.org