Re: [PR] [XERCESJ-1781] Javadoc fixes [xerces-j]

elharo (via GitHub) <[email protected]>
Newsgroups gmane.text.xml.xerces-j.devel
Message-ID <PR_kwDOLzdO6M6xxdNC-9850d373-654e-4896-aa08-067f2da6a1f0@gitbox.apache.org>
elharo commented on code in PR #44:
URL: https://github.com/apache/xerces-j/pull/44#discussion_r2507105970


##########
src/org/apache/xerces/jaxp/datatype/XMLGregorianCalendarImpl.java:
##########
@@ -48,19 +48,18 @@
  * defined in the XML Namespace
  * <code>"http://www.w3.org/2001/XMLSchema"</code>.
  * These datatypes are normatively defined in 
- * <a href="http://www.w3.org/TR/xmlschema-2/#dateTime">W3C XML Schema 1.0 Part 2, Section 3.2.7-14</a>.</p>
+ * <a href="http://www.w3.org/TR/xmlschema-2/#dateTime">W3C XML Schema 1.0 Part 2, Section 3.2.7-14</a>.
  *
- * <p>The table below defines the mapping between XML Schema 1.0
- * date/time datatype fields and this class' fields. It also summarizes
- * the value constraints for the date and time fields defined in
- * <a href="http://www.w3.org/TR/xmlschema-2/#isoformats">W3C XML Schema 1.0 Part 2, Appendix D,
- * <i>ISO 8601 Date and Time Formats</i></a>.</p>
+ * <p>The table below defines the mapping between XML Schema 1.0 date/time datatype fields and this class' fields.
+ * It also summarizes the value constraints for the date and time fields defined in
+ * <a href="http://www.w3.org/TR/xmlschema-2/#isoformats">W3C XML Schema 1.0 Part 2, Appendix D, <i>ISO 8601 Date and Time Formats</i></a>.
+ * </p>
  * 
- * <a name="datetimefieldsmapping"/>
- * <table border="2" rules="all" cellpadding="2">
+ * <a name="datetimefieldsmapping"></a>
+ * <table border="1">
  *   <thead>
  *     <tr>
- *       <th align="center" colspan="3">
+ *       <th colspan="3">

Review Comment:
   that's fine, but it doesn't seem to be here yet. Do you need to push?



-- 
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]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.