SingingBush commented on code in PR #74:
URL: https://github.com/apache/xerces-j/pull/74#discussion_r2590690807
##########
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:
Not here but it's abstract class, so expect that other classes will override
so want the docs to make sure they make sense when 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]
