On 10/30/2013 2:58 AM, Daniel Fuchs wrote:
On 10/30/13 1:14 AM, huizhe wang wrote:
I updated the webrev to also fix the error message that showed the
actual number of attributes parsed rather than the limit itself.
http://cr.openjdk.java.net/~joehw/jdk8/8024378/webrev/
Thanks,
Joe
Hi Joe,
Looks good to me. I see another change, which is that now reset() will
also do:
fInScanContent = false;
But by the look of it I suspect it was a bug that it did not reset it
previously?
Yes. But since fInScanContent is always being reset to false after a
scan, this is actually redundant. It was previously in
reset(XMLComponentManager...) but not reset(PropertyManager ...).
best,
Joe
best regards,
-- daniel
On 10/29/2013 4:49 PM, huizhe wang wrote:
It appears the previous patch was one line short of supporting setting
the JAXP's new property ElementAttributeLimit through the StAX
factory, that is, resetting the variable in reset(PropertyManager
propertyManager) as well. Other than that, I also moved the common
reset items from reset(XMLComponentManager...) and
reset(PropertyManager ...) to a new method called "resetCommon".
webrevs: http://cr.openjdk.java.net/~joehw/jdk8/8024378/webrev/
The real change is to set the following in both reset methods, now in
resetCommon:
663 fElementAttributeLimit =
fSecurityManager.getLimit(XMLSecurityManager.Limit.ELEMENT_ATTRIBUTE_LIMIT);
Thanks,
Joe