[jira] Commented: (COCOON3-16) Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer

2009-01-26 Thread Grzegorz Kossakowski (JIRA)

[ 
https://issues.apache.org/jira/browse/COCOON3-16?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12667459#action_12667459
 ] 

Grzegorz Kossakowski commented on COCOON3-16:
-

Steven, I can see some other changes to SaxBuffer. Are they formatting-only?

 Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer
 

 Key: COCOON3-16
 URL: https://issues.apache.org/jira/browse/COCOON3-16
 Project: Cocoon 3
  Issue Type: Bug
  Components: cocoon-pipeline
Affects Versions: 3.0.0-alpha-1
Reporter: Steven Dolg
Assignee: Steven Dolg
 Fix For: 3.0.0-alpha-2

 Attachments: consumer.patch


 The interface o.a.c.p.c.Consumer does not extend o.a.c.p.c.PipelineComponent.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (COCOON3-16) Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer

2009-01-26 Thread Andreas Pieber (JIRA)

[ 
https://issues.apache.org/jira/browse/COCOON3-16?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12667468#action_12667468
 ] 

Andreas Pieber commented on COCOON3-16:
---

Looks quite nice, but wouldn't it better to inherit the following classes from 
AbstractPipelineComponent?

o.a.c.p.u.SaxBuffer
o.a.c.p.u.NullXMLConsumer

 Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer
 

 Key: COCOON3-16
 URL: https://issues.apache.org/jira/browse/COCOON3-16
 Project: Cocoon 3
  Issue Type: Bug
  Components: cocoon-pipeline
Affects Versions: 3.0.0-alpha-1
Reporter: Steven Dolg
Assignee: Steven Dolg
 Fix For: 3.0.0-alpha-2

 Attachments: consumer.patch


 The interface o.a.c.p.c.Consumer does not extend o.a.c.p.c.PipelineComponent.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (COCOON3-16) Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer

2009-01-26 Thread Steven Dolg (JIRA)

[ 
https://issues.apache.org/jira/browse/COCOON3-16?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12667465#action_12667465
 ] 

Steven Dolg commented on COCOON3-16:


Yes, just checked that - sorry for that.

I guess I'll make a patch without formatting changes...

 Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer
 

 Key: COCOON3-16
 URL: https://issues.apache.org/jira/browse/COCOON3-16
 Project: Cocoon 3
  Issue Type: Bug
  Components: cocoon-pipeline
Affects Versions: 3.0.0-alpha-1
Reporter: Steven Dolg
Assignee: Steven Dolg
 Fix For: 3.0.0-alpha-2

 Attachments: consumer.patch


 The interface o.a.c.p.c.Consumer does not extend o.a.c.p.c.PipelineComponent.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (COCOON3-16) Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer

2009-01-26 Thread Steven Dolg (JIRA)

[ 
https://issues.apache.org/jira/browse/COCOON3-16?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12667531#action_12667531
 ] 

Steven Dolg commented on COCOON3-16:


Actually I'm not sure about those classes inheriting from 
AbstractPipelineComponent.

While technically it would make no difference (no additional interfaces 
implemented nor anything else), this somehow appears strange to me.
Mostly because those are no real pipeline components.

I guess this will need some more changes/improvements, but currently I have no 
idea what or how...

 Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer
 

 Key: COCOON3-16
 URL: https://issues.apache.org/jira/browse/COCOON3-16
 Project: Cocoon 3
  Issue Type: Bug
  Components: cocoon-pipeline
Affects Versions: 3.0.0-alpha-1
Reporter: Steven Dolg
Assignee: Steven Dolg
 Fix For: 3.0.0-alpha-2

 Attachments: consumer.patch


 The interface o.a.c.p.c.Consumer does not extend o.a.c.p.c.PipelineComponent.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (COCOON3-16) Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer

2009-01-26 Thread Carsten Ziegeler (JIRA)

[ 
https://issues.apache.org/jira/browse/COCOON3-16?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12667598#action_12667598
 ] 

Carsten Ziegeler commented on COCOON3-16:
-

We should use the SAXBuffer from cocoon-xml instead and do we really need 
NullXMLConsumer?
I think we can go without an XMLConsumer completly and just really on 
ContentHandler. I'll commit with an RT about that in the list.

 Fix the class hierarchy of org.apache.cocoon.pipeline.component.Consumer
 

 Key: COCOON3-16
 URL: https://issues.apache.org/jira/browse/COCOON3-16
 Project: Cocoon 3
  Issue Type: Bug
  Components: cocoon-pipeline
Affects Versions: 3.0.0-alpha-1
Reporter: Steven Dolg
Assignee: Steven Dolg
 Fix For: 3.0.0-alpha-2

 Attachments: consumer.patch


 The interface o.a.c.p.c.Consumer does not extend o.a.c.p.c.PipelineComponent.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.