subject:"git commit\: \[CXF\-5866\] Adding XMLStreamReaderWrapper to WSDLManagerImpl and providing a XMLStreamReader wrapper that resolves system properties"

Re: git commit: [CXF-5866] Adding XMLStreamReaderWrapper to WSDLManagerImpl and providing a XMLStreamReader wrapper that resolves system properties

2014-07-10 Thread Daniel Kulp

Alessio, Two thoughts: 1) There is a java.xml.stream.util.StreamReaderDelegate class available from StAX apis, is there really a need for another delegate in CXF? 2) Would it make sense to change the SysPropExpandingStreamReader to a more generic “PropertiesExpandingStreamReader” that takes

Re: git commit: [CXF-5866] Adding XMLStreamReaderWrapper to WSDLManagerImpl and providing a XMLStreamReader wrapper that resolves system properties

2014-07-10 Thread Jason Pell

That definitely sounds good. I am so sick of system properties for configuration. On 10/07/2014 10:30 PM, Daniel Kulp dk...@apache.org wrote: Alessio, Two thoughts: 1) There is a java.xml.stream.util.StreamReaderDelegate class available from StAX apis, is there really a need for another