classpath-patches
[Top][All Lists]
Advanced

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

Re: [cp-patches] [Patch] java.nio cleanup


From: Mark Wielaard
Subject: Re: [cp-patches] [Patch] java.nio cleanup
Date: Fri, 15 Oct 2004 12:00:30 +0200

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.

Cheers,

Mark

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


reply via email to

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