Return-Path: X-Original-To: apmail-commons-issues-archive@minotaur.apache.org Delivered-To: apmail-commons-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 945739B29 for ; Sat, 15 Oct 2011 18:33:35 +0000 (UTC) Received: (qmail 61704 invoked by uid 500); 15 Oct 2011 18:33:35 -0000 Delivered-To: apmail-commons-issues-archive@commons.apache.org Received: (qmail 61644 invoked by uid 500); 15 Oct 2011 18:33:35 -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 61636 invoked by uid 99); 15 Oct 2011 18:33:35 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 15 Oct 2011 18:33:35 +0000 X-ASF-Spam-Status: No, hits=-2000.5 required=5.0 tests=ALL_TRUSTED,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 15 Oct 2011 18:33:34 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id 343B530903C for ; Sat, 15 Oct 2011 18:33:14 +0000 (UTC) Date: Sat, 15 Oct 2011 18:33:14 +0000 (UTC) From: "Daniel Pitts (Created) (JIRA)" To: issues@commons.apache.org Message-ID: <829172049.17510.1318703594215.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Created] (OGNL-27) Move "toString" implementations into visitor pattern. MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 Move "toString" implementations into visitor pattern. ----------------------------------------------------- Key: OGNL-27 URL: https://issues.apache.org/jira/browse/OGNL-27 Project: OGNL Issue Type: New Feature Reporter: Daniel Pitts Using the Visitor pattern allows for a cleaner implementation of toString(). I have a patch which will remove toString() from all AST classes, and replace it with a single toString() in "SimpleNode" which delegates to a ToStringVisitor to build the String efficiently. This patch can also be used as an example of how to move other business logic out of the AST classes into their own visitor classes. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira