classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] java.util Documentation Patch 02


From: Andrew John Hughes
Subject: Re: [cp-patches] java.util Documentation Patch 02
Date: Sun, 05 Sep 2004 20:13:40 +0100

On Sun, 2004-09-05 at 19:58, Mark Wielaard wrote:
> Hi,
> 
> Deep inside your documentation patch (thanks for that again!) I found
> this:
> 
> On Mon, 2004-08-16 at 03:02, Andrew John Hughes wrote:
> > --- java/util/AbstractList.java 7 May 2002 05:13:05 -0000       1.21
> > +++ java/util/AbstractList.java 16 Aug 2004 00:57:59 -0000
> > @@ -85,7 +85,7 @@ public abstract class AbstractList exten
> >     * <code>add(int, Object)</code> and <code>remove(int)</code> methods.
> >     * Otherwise, this field may be ignored.
> >     */
> > -  protected int modCount;
> > +  protected transient int modCount;
> 
> Was this change deliberately?
> You didn't mention the change in the ChangeLog.
> Are you sure this is OK? It does make sense to make it transient since
> it should be cleared after (de)serialization. But it does change the
> serialization of classes extending AbstractList.
> 
> Cheers,
> 
> Mark

Whoops, yes sorry -- the change was deliberate but I forgot to add it to
the ChangeLog.  The new behaviour matches Sun's implementation, which
has modCount as transient too.  I will add it to the ChangeLog.
-- 
Andrew :-)
 
Please avoid sending me Word or PowerPoint attachments.
See http://www.fsf.org/philosophy/no-word-attachments.html
 
Value your freedom, or you will lose it, teaches history.
`Don't bother us with politics' respond those who don't want to learn.
 

Attachment: signature.asc
Description: This is a digitally signed message part


reply via email to

[Prev in Thread] Current Thread [Next in Thread]