Return-Path: Delivered-To: apmail-maven-doxia-dev-archive@www.apache.org Received: (qmail 6537 invoked from network); 3 Aug 2007 17:57:13 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 3 Aug 2007 17:57:13 -0000 Received: (qmail 13992 invoked by uid 500); 3 Aug 2007 17:57:13 -0000 Delivered-To: apmail-maven-doxia-dev-archive@maven.apache.org Received: (qmail 13964 invoked by uid 500); 3 Aug 2007 17:57:13 -0000 Mailing-List: contact doxia-dev-help@maven.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: doxia-dev@maven.apache.org Delivered-To: mailing list doxia-dev@maven.apache.org Received: (qmail 13955 invoked by uid 99); 3 Aug 2007 17:57:13 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 03 Aug 2007 10:57:13 -0700 X-ASF-Spam-Status: No, hits=1.2 required=10.0 tests=SPF_NEUTRAL X-Spam-Check-By: apache.org Received-SPF: neutral (athena.apache.org: local policy) Received: from [82.209.166.4] (HELO smtp.bredband2.net) (82.209.166.4) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 03 Aug 2007 17:57:08 +0000 Received: (qmail 346 invoked from network); 3 Aug 2007 17:52:54 -0000 Received: from me-24-111-233-83.3.cust.bredband2.com (HELO [192.168.0.3]) ([83.233.111.24]) (envelope-sender ) by smtp.bredband2.net (qmail-ldap-1.03) with SMTP for ; 3 Aug 2007 17:52:54 -0000 Message-ID: <46B36C57.60403@apache.org> Date: Fri, 03 Aug 2007 19:56:39 +0200 From: Dennis Lundberg User-Agent: Thunderbird 2.0.0.6 (Windows/20070728) MIME-Version: 1.0 To: doxia-dev@maven.apache.org Subject: Re: Doxia core clean up References: <9ae367340708030529t63dd6431n7c40705bfda57751@mail.gmail.com> In-Reply-To: <9ae367340708030529t63dd6431n7c40705bfda57751@mail.gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org Vincent Siveton wrote: > Hi, > > Several utility classes in Doxia are few used or unused. > org.apache.maven.doxia.util.StringUtil (used only in > XmlWriterXdocSink, which is not maintained) > org.apache.maven.doxia.util.FileUtil (used only in DocBookSink) > org.apache.maven.doxia.util.WrappedException (unused) > > I propose to remove them and to use Plexus-utils instead of. It would > be a part of DOXIA-56. Sounds good to me. > Also, I propose to rename the following package > org.apache.maven.doxia.site.module.manager > to > org.apache.maven.doxia.module.manager What would be the benefit of that move? The JavaDoc says that it is a hook for doxia site tools, so the package "site" makes sense to me. Perhaps change it to org.apache.maven.doxia.sitetools.module.manager instead? > WDYT? > > Vincent -- Dennis Lundberg