maven-doxia-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dennis Lundberg <denn...@apache.org>
Subject Re: Doxia core clean up
Date Fri, 03 Aug 2007 17:56:39 GMT
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

Mime
View raw message