elharo commented on code in PR #74:
URL: https://github.com/apache/xerces-j/pull/74#discussion_r2590760900
##########
src/org/apache/xerces/parsers/AbstractSAXParser.java:
##########
@@ -2151,7 +2158,9 @@ protected final void startNamespaceMapping() throws
SAXException{
}
/**
- * Send endPrefixMapping events
+ * Send endPrefixMapping events.
+ *
+ * @throws SAXException if an error occurs
Review Comment:
I don't think this can be thrown either
##########
src/org/apache/xerces/parsers/AbstractSAXParser.java:
##########
@@ -2127,14 +2129,19 @@ protected void setLexicalHandler(LexicalHandler handler)
* Returns the lexical handler.
*
* @see #setLexicalHandler
+ * @return the lexical handler
+ * @throws SAXNotRecognizedException not thrown by the abstract SAX parser
+ * @throws SAXNotSupportedException not thrown by the abstract SAX parser
*/
protected LexicalHandler getLexicalHandler()
throws SAXNotRecognizedException, SAXNotSupportedException {
return fLexicalHandler;
} // getLexicalHandler():LexicalHandler
/**
- * Send startPrefixMapping events
+ * Send startPrefixMapping events.
+ *
+ * @throws SAXException if an error occurs
Review Comment:
abstract class but final method, so not overridden
--
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]
