classpath-patches
[Top][All Lists]
Advanced

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

Re: [cp-patches] Needs Approval: Patch - JList multiple selection includ


From: Tom Tromey
Subject: Re: [cp-patches] Needs Approval: Patch - JList multiple selection including Shift-Click
Date: 01 Jul 2005 10:36:20 -0600
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50

>>>>> "Roman" == Roman Kennke <address@hidden> writes:

Roman> - No need to make every field private. This could impact performance in
Roman> some cases (like when inner classes access the private fields of
Roman> enclosing classes, the compiler has to generate accessor methods for
Roman> them).

Ordinarily I enable the eclipse warning for this and then I make a
given member package-private if it is actually used from an inner
class.  I also add a comment explaining this.

Tom




reply via email to

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