> Paul / Tom: does this mean I should not attempt to check this patch in for
> now?  It looks great to me with a small modification to put the if()'s into
> the case (2 less function calls and no readability loss IMO).  John: you OK
> with that change?

Yes that's perfect.

John Leuner
 
> --John
> 
> ----- Original Message -----
> From: "Tom Tromey" <[EMAIL PROTECTED]>
> To: "John Leuner" <[EMAIL PROTECTED]>
> Cc: "John Keiser" <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]>
> Sent: Wednesday, April 25, 2001 11:05 PM
> Subject: Re: Problem in Class.java
> 
> 
> > >>>>> "John" == John Leuner <[EMAIL PROTECTED]> writes:
> >
> > John> BTW, is this the correct way to generate and submit a patch?
> >
> > Yes, thanks.
> >
> > This patch is large enough to require paperwork.  I don't know whether
> > you have already filed it or not.  (Even once it is in I probably
> > won't approve or reject this patch, as I'm not familiar with this part
> > of Classpath.)
> >
> > Tom
> 
> 
> _______________________________________________
> Classpath mailing list
> [EMAIL PROTECTED]
> http://mail.gnu.org/mailman/listinfo/classpath

-- 

_______________________________________________
Classpath mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/classpath

Reply via email to