Re: RFR (jaxp): 8016133 : Regression: diff. behavior with user-defined SAXParser

2013-06-17 Thread Daniel Fuchs
Hi Joe, Looks good to me. Not a reviewer - as you know ;-) -- daniel On 6/11/13 5:36 AM, huizhe wang wrote: Hi, This is a quick fix on a regression caused by a previous patch. The XMLReaderFactory uses a class variable (_jarread) to indicate if service file has already been read. Along with

Re: RFR (jaxp): 8016133 : Regression: diff. behavior with user-defined SAXParser

2013-06-17 Thread Chris Hegarty
Looks ok to me too Joe. -Chris. On 17/06/2013 09:17, Daniel Fuchs wrote: Hi Joe, Looks good to me. Not a reviewer - as you know ;-) -- daniel On 6/11/13 5:36 AM, huizhe wang wrote: Hi, This is a quick fix on a regression caused by a previous patch. The XMLReaderFactory uses a class

RFR (jaxp): 8016133 : Regression: diff. behavior with user-defined SAXParser

2013-06-10 Thread huizhe wang
Hi, This is a quick fix on a regression caused by a previous patch. The XMLReaderFactory uses a class variable (_jarread) to indicate if service file has already been read. Along with this variable, there was another (clsFromJar ) that caches the classname if found in a service file. The 2nd