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

Reply via email to