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 Tue, 14 Aug 2007 23:07:22 GMT
Jason,

Any comments?

Cheers,

Vincent

2007/8/3, Vincent Siveton <vincent.siveton@gmail.com>:
> 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