This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Addendum (Re: Patch: FYI: clone System properties)


Ranjit> The following patch is an addendum to your patch. It implements
Ranjit> clone( ) for Properties and also attempts to address Classpath
Ranjit> support request number 100559 raised by crawley and pointed out
Ranjit> by Mike that lists the differences in the output of
Ranjit> Properties.list( ) in GCJ v/s Sun's JDK 1.3/1.4:

I thought about this more, and I think we don't need a clone method,
due to the way that Hashtable.clone is implemented.  I'm going to
leave that part out of your patch.
Can you please explain the reasoning? AFAICT, without this clone()
implementation, Properties.clone( ) would give you an object
that doesn't have the same behaviour with respect to default values
for keys. The JDK docs do not say anything about Properties.clone( ),
but this is the behaviour I think it ought to have.

Ranjit.

_________________________________________________________________
The new MSN 8: advanced junk mail protection and 2 months FREE* http://join.msn.com/?page=features/junkmail


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]