Return-Path: Delivered-To: apmail-commons-issues-archive@minotaur.apache.org Received: (qmail 89316 invoked from network); 2 Apr 2010 23:43:49 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 2 Apr 2010 23:43:49 -0000 Received: (qmail 56063 invoked by uid 500); 2 Apr 2010 23:43:49 -0000 Delivered-To: apmail-commons-issues-archive@commons.apache.org Received: (qmail 55909 invoked by uid 500); 2 Apr 2010 23:43:49 -0000 Mailing-List: contact issues-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: issues@commons.apache.org Delivered-To: mailing list issues@commons.apache.org Received: (qmail 55901 invoked by uid 99); 2 Apr 2010 23:43:48 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 02 Apr 2010 23:43:48 +0000 X-ASF-Spam-Status: No, hits=-1208.9 required=10.0 tests=ALL_TRUSTED,AWL X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 02 Apr 2010 23:43:48 +0000 Received: from brutus.apache.org (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id BD34D234C1EF for ; Fri, 2 Apr 2010 23:43:27 +0000 (UTC) Message-ID: <1807079506.668511270251807773.JavaMail.jira@brutus.apache.org> Date: Fri, 2 Apr 2010 23:43:27 +0000 (UTC) From: "Rory Winston (JIRA)" To: issues@commons.apache.org Subject: [jira] Updated: (NET-318) Article.references field is never updated; extremely inefficient implementation In-Reply-To: <1477423368.611931270060287174.JavaMail.jira@brutus.apache.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/NET-318?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Rory Winston updated NET-318: ----------------------------- Priority: Minor (was: Major) > Article.references field is never updated; extremely inefficient implementation > ------------------------------------------------------------------------------- > > Key: NET-318 > URL: https://issues.apache.org/jira/browse/NET-318 > Project: Commons Net > Issue Type: Bug > Affects Versions: 2.0 > Reporter: Sebb > Priority: Minor > > Article.references is a private StringBuffer instance. > The method addReference(String) inserts the reference into the buffer followed by a tab. > However, as far as I can tell, this is never called. > The method getReferences() - which is used - has to parse the StringBuffer in order to find the references. > Since the buffer is only used in the add and get methods, it does not make sense to handle the references this way. > Even if the buffer were used (e.g. as part of a toString() method) it would still be better to store the references as a List or Array. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.