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: Tom Tromey
Subject: Re: [cp-patches] Patch: FYI: fix for constructor oddity
Date: 19 Sep 2004 15:51:58 -0600
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50

>>>>> "Mark" == Mark Wielaard <address@hidden> writes:

Mark> Agreed. But now it has a public constructor which means that anybody
Mark> could make an instance of ProcessThread.

Only people who can access the type, which is private.
Or perhaps you mean at runtime -- that's true, you could do it via
reflection.

Mark> So please make this a package
Mark> local contructor.

Ok.

gcjx has detected a *lot* of problems like this in Classpath.  I will
probably write a jumbo patch to fix them all.  IMO it should be part
of our coding standard to avoid this situation.

Tom




reply via email to

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