[kaffe] Kaffe CVS: kaffe dalibor

Dalibor Topic robilad@yahoo.com
Sat, 25 May 2002 06:46:15 -0700 (PDT)


Hi Archie,

--- Archie Cobbs <archie@dellroad.org> wrote:
> Kaffe CVS writes:
> > Changes by:	dalibor	02/05/24 07:46:43
> > 
> > Modified files:
> > 	.              : ChangeLog 
> > 	libraries/javalib/java/util: HashMap.java
> TreeMap.java 
> > 
> > Log message:
> > cleaned up clone in java.util.TreeMap and
> java.util.HashMap
> 
> Can you explain the purpose of these commits? I
> don't see the
> difference.

I ran jlint over kaffe's class libraries and it
complained about clone being NULL if super.clone()
failed, and it being used to set instance fields
nevertheless.

Despite the comment that says "can't happen" I felt it
was safer to move the code working on the cloned
object inside the try/catch block.

cheers,

dalibor topic

__________________________________________________
Do You Yahoo!?
Yahoo! - Official partner of 2002 FIFA World Cup
http://fifaworldcup.yahoo.com