Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cocoon.apache.org
Message-ID: <495E1027.2090506@apache.org>
Date: Fri, 02 Jan 2009 14:01:27 +0100
From: Carsten Ziegeler <cziegeler@apache.org>
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; de;
 rv:1.8.1.19) Gecko/20081209 Lightning/0.9 Thunderbird/2.0.0.19
 Mnenhy/0.7.5.666
MIME-Version: 1.0
To: dev@cocoon.apache.org
Subject: Re: [C3] Extract SAX from the Pipeline module
References: <495E08F5.9010907@indoqa.com>
In-Reply-To: <495E08F5.9010907@indoqa.com>
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: 7bit

Steven Dolg wrote:
> Hello,
> 
> as indicated some time ago, I would like to remove all the SAX related
> pipeline components from the Pipeline module and put them into their own
> module.
> 
> Now I have done this locally just to see how easy or difficult this
> would be.
> I found that some modules (e.g. cocoon-servlet) actually rely on those
> SAX components.
> This means the dependencies amongst the modules will change a little bit
> and the SAX module will actually be a dependency for some other
> higher-level modules. (But I find nothing wrong with that).
> But this also means the patch will be rather huge and affect a lot of
> the existing modules.
> 
> I intend to finish this and provide a patch in the next 1-2 weeks.
> If someone is currently working on some components or modules this might
> lead to a merge-mess.
> So let me know if this might be a problem for you.
> 
> 
> As second step I'd like to rename some of those components to clearly
> indicate that they're handling XML using SAX - as opposed to our new
> components using StAX.
> But I guess that can (and should) wait until the new module is there.
> 
> 
> Any opinions or objections on that?
Sounds great!

The only comment I have atm is about the naming of the sax classes,
I think for consistency we should use upper case characters (SAX) as
this is the way the classes are named in the jdk (same with DOM).

Carsten
-- 
Carsten Ziegeler
cziegeler@apache.org