classpath-patches
[Top][All Lists]
Advanced

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

Re: [cp-patches] Patch: FYI: fix for constructor oddity


From: Mark Wielaard
Subject: Re: [cp-patches] Patch: FYI: fix for constructor oddity
Date: Sat, 18 Sep 2004 10:27:51 +0200

Hi,

On Sat, 2004-09-18 at 03:52, Tom Tromey wrote:
> I'm checking this in on the trunk.  This fixes a small oddity in
> VMProcess, which I think is adequately well explained by the comment I
> added.
> 
> Basically, use of "private" is strange sometimes, it is best to avoid
> it when you have accesses that cross inner/outer boundaries.

Agreed. But now it has a public constructor which means that anybody
could make an instance of ProcessThread. So please make this a package
local contructor.

Thanks,

Mark

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


reply via email to

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