maven-doxia-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vincent Siveton" <vincent.sive...@gmail.com>
Subject Re: Doxia core clean up
Date Fri, 03 Aug 2007 21:43:19 GMT
Hi,

2007/8/3, Jason van Zyl <jason@maven.org>:
> +1
>
> What about the other changes you were going to make? Are we going to
> leave the names as "Parser" and not "Markup".

If I understand you as well, you recommend to put all constants
defined in Markup interfaces in AbstractParser, AbstractTextParser and
AbstractXmlParser and other XXXParser.

My only point is that XXXSink should uses XXXParser.CONSTANTS inside. Thoughts?

Cheers,

Vincent

> And did you get anywhere using the single Plexus JAR?
>
> On 3 Aug 07, at 8:29 AM 3 Aug 07, 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.
> >
> > Also, I propose to rename the following package
> > org.apache.maven.doxia.site.module.manager
> > to
> > org.apache.maven.doxia.module.manager
> >
> > WDYT?
> >
> > Vincent
>
> Thanks,
>
> Jason
>
> ----------------------------------------------------------
> Jason van Zyl
> Founder and PMC Chair, Apache Maven
> jason at sonatype dot com
> ----------------------------------------------------------
>
>
>
>

Mime
View raw message