SingingBush commented on code in PR #28:
URL: https://github.com/apache/xerces-j/pull/28#discussion_r2230525473
##########
src/org/apache/xerces/xni/QName.java:
##########
@@ -71,12 +72,21 @@ public QName() {
clear();
} // <init>()
- /** Constructs a QName with the specified values. */
+ /**
+ * Constructs a QName with the specified values.
+ * @param prefix The qname prefix. (e.g. "a")
+ * @param localpart The qname localpart. (e.g. "foo")
Review Comment:
for this one I think the description from the same params that already
existed on another method. I don't mind changing it in both places though.
Could you suggest an alternative.
--
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: j-dev-unsubscr...@xerces.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: j-dev-unsubscr...@xerces.apache.org
For additional commands, e-mail: j-dev-h...@xerces.apache.org
