jkesselm commented on code in PR #1: URL: https://github.com/apache/xalan-test/pull/1#discussion_r1189162388
########## tests/bugzilla/Bugzilla1251.java: ########## @@ -37,6 +37,19 @@ * Testlet for reproducing Bugzilla reported bugs. * * Reported-by: sl...@matavnet.hu + * + * 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: Sorry, I write in 8-bit ASCII unless there is good reason not to. Suggestion declined unless there is a W3C style guide mandating otherwise. ########## tests/bugzilla/Bugzilla1251.java: ########## @@ -37,6 +37,19 @@ * Testlet for reproducing Bugzilla reported bugs. * * Reported-by: sl...@matavnet.hu + * + * 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: Sorry, I write in 7-bit ASCII unless there is good reason not to. Suggestion declined unless there is a W3C style guide mandating otherwise. -- 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: dev-unsubscr...@xalan.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@xalan.apache.org For additional commands, e-mail: dev-h...@xalan.apache.org
