[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] [Patch] java.nio cleanup
From: |
Michael Koch |
Subject: |
Re: [cp-patches] [Patch] java.nio cleanup |
Date: |
Fri, 15 Oct 2004 12:21:11 +0200 |
User-agent: |
KMail/1.6.2 |
On Friday 15 October 2004 12:07, Jeroen Frijters wrote:
> Michael Koch wrote:
> > On Friday 15 October 2004 12:00, Mark Wielaard wrote:
> > > Hi,
> > >
> > > On Fri, 2004-10-15 at 09:27, Michael Koch wrote:
> > > > I just commited the attached patch to cleanup java.nio a bit.
> > > > Import statements and small coding style nits.
> > > > [...]
> > > > --- java/nio/MappedByteBuffer.java 8 Apr 2004 21:25:59
> > > > -0000 1.11
> > > > +++ java/nio/MappedByteBuffer.java 15 Oct 2004 07:26:46
> > > > -0000 [...]
> > > > - public void finalize () { unmapImpl(); }
> > > > + public void finalize()
> > > > + throws Throwable
> > > > + {
> > > > + unmapImpl();
> > > > + }
> > > > }
> > >
> > > Why is this finalize method public and not protected?
> > > And why does it need to throw Throwable?
> > > Any reason to have this finalize method at all in this class.
> > > It seems that finalize(), unmapImpl() and forceImpl() should
> > > only be defined in MappedByteBufferImpl.
> >
> > Don't know. Ask SUN. It's in the API this way.
>
> Huh? Not in my copy.
Damn, japitools fooled me.
I will change this code again.
Michael