Return-Path: Delivered-To: apmail-harmony-commits-archive@www.apache.org Received: (qmail 46288 invoked from network); 21 Dec 2006 16:00:43 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 21 Dec 2006 16:00:43 -0000 Received: (qmail 6119 invoked by uid 500); 21 Dec 2006 16:00:50 -0000 Delivered-To: apmail-harmony-commits-archive@harmony.apache.org Received: (qmail 6097 invoked by uid 500); 21 Dec 2006 16:00:49 -0000 Mailing-List: contact commits-help@harmony.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@harmony.apache.org Delivered-To: mailing list commits@harmony.apache.org Received: (qmail 6087 invoked by uid 99); 21 Dec 2006 16:00:49 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 21 Dec 2006 08:00:49 -0800 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO brutus.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 21 Dec 2006 08:00:41 -0800 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 4C06971429F for ; Thu, 21 Dec 2006 08:00:21 -0800 (PST) Message-ID: <16643870.1166716821308.JavaMail.jira@brutus> Date: Thu, 21 Dec 2006 08:00:21 -0800 (PST) From: "Sian January (JIRA)" To: commits@harmony.apache.org Subject: [jira] Created: (HARMONY-2840) [classlib] [luni] Some inner classes could be removed or changed to static inner classes (FindBugs) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [classlib] [luni] Some inner classes could be removed or changed to static inner classes (FindBugs) --------------------------------------------------------------------------------------------------- Key: HARMONY-2840 URL: http://issues.apache.org/jira/browse/HARMONY-2840 Project: Harmony Issue Type: Bug Components: Classlib Reporter: Sian January Priority: Minor A static inner class takes up less space in memory, so if an inner class doesn't use the reference to the parent object it makes sense to turn it into a static inner class. I have also found a couple of instances where an inner class has been used as a lock object, but there didn't seem to be any reason why the synchronization wasn't done on the parent object, so this bug also addresses that. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira