Jason, Any comments?
Cheers, Vincent 2007/8/3, Vincent Siveton <[EMAIL PROTECTED]>: > Hi, > > 2007/8/3, Jason van Zyl <[EMAIL PROTECTED]>: > > +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 > > ---------------------------------------------------------- > > > > > > > > >
