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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
