avalon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeremias Maerki <dev.jerem...@greenmail.ch>
Subject Re: Refactoring of Excalibur XMLUtil
Date Wed, 15 Jan 2003 08:30:38 GMT

On 15.01.2003 09:22:49 Carsten Ziegeler wrote:
> > I'm currently trying to make XML-Forrest compile again after updating
> > Cocoon from CVS. The reason was the patch for bug #13247.
> > 
> > You've split up the Parser interface in two, one in the sax subpackage
> > and one in the dom subpackage. My personal opinion would have been to
> > rename the interfaces to SAXParser and DOMParser to avoid having to use
> > fully qualified classnames because of the same name.
> > 
> Yes, I thought of that, too - and wanted to do it today. (To be precise,
> it wasn't me that started this splitting, but that's not important).

Sorry. Next time I'll have a closer look.

> > Also, wouldn't it make sense to move the default implementations to a
> > separate package? They are now in the sax subpackage and reference the
> > dom package. I would seem clearer to me to move them to perhaps
> > org.apache.excalibur.xml.impl.
> > 
> Hey, twilight zone...yes, this is the second change, I wanted to do
> today.

:-)

> > What do you think?
> > 
> I didn't had time yesterday, to do all of this on one day.
> 
> So, I think it's a very good idea and I will do it asap.

I'll watch CVS then. Thanks!

(Congratulations to your article on Avalon in the "Javamagazin" by the
way!)

Jeremias Maerki
FOP committer

--
To unsubscribe, e-mail:   <mailto:avalon-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:avalon-dev-help@jakarta.apache.org>


Mime
View raw message