jkesselm commented on code in PR #1: URL: https://github.com/apache/xalan-test/pull/1#discussion_r1189207477
########## tests/bugzilla/Bugzilla1251.java: ########## @@ -37,6 +37,19 @@ * Testlet for reproducing Bugzilla reported bugs. * * Reported-by: [email protected] + * + * jkesselm May2023: CONFIRMED OPPOSITE. Despite claim from user, identity transformer reports FAIL (throws exception), + * real transformer reports PASS (exception caught and handled as intended). + * + * In real transformer, ElemCopy catches the SaxException, throws it wrapped in TransformerException; + * TransformerImpl catches that, sees that it has an m_serializationHandler. and passes it off to + * that. Since that reports checkPass, test is considered good. + * + * In TransformerIdentityImpl.startElement(), there is no equivalent catch-and-handle for exceptions. + * + * RECOMMENDATION: SANITY CHECK. User appears to have misstated the problem. If so -- if the description is Review Comment: Change pushed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
