Jeremias Maerki wrote:
> 
> Hi Carsten
> 
> 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).

> 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.

Carsten

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to