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

Reply via email to