On 12/18/12 1:09 AM, Paul Sandoz wrote:
On Dec 18, 2012, at 7:02 AM, Mandy Chung<mandy.ch...@oracle.com> wrote:
Hi Daniel,
There are currently several different FactoryFinder class in the JAXP
implementation and there might be slight difference in each of them. Do you
know if it has been looked into whether it's feasible to merge them into a
single utility class? I suspect someone might have brought this up in the past.
You suspect right :-)
Those FactoryFinder classes are package private, probably a artefact of JAXP
having multiple distribution channels, and there were subtle differences
between them but perhaps less so now?
IMHO such unification could be considered as a later fix as i think getting the
SL and CL functionality correct is a higher priority.
Agree and such unification can be done separately.
http://cr.openjdk.java.net/~dfuchs/JDK-7169894/javax.xml.stream/webrev.01/
Daniel - I reviewed the change and also the latest webrev for
parser/stream. They look good to me.
Thanks
Mandy
