On Fri, 10 Jan 2025 14:43:00 GMT, Kevin Walls <[email protected]> wrote:
> DescriptorSupport has a constructor and a method providing creation from, and
> export to, XML.
>
> These are unused in the JDK and have no practical known examples of usage.
> XML parsing is best done by an independent implementation, not this class.
src/java.management/share/classes/javax/management/modelmbean/DescriptorSupport.java
line 205:
> 203: * @exception MBeanException Wraps a distributed communication
> Exception.
> 204: * @deprecated This constructor exists for historical reasons. It
> enables
> 205: * reading from XML, which is unnecessary.
I wonder if we should say instead "Reading from XML if required should be
implemented externally".
src/java.management/share/classes/javax/management/modelmbean/DescriptorSupport.java
line 975:
> 973: * thrown.
> 974: * @deprecated This method exists for historical reaons. It
> 975: * enables writing as XML, which is unnecessary.
Same here: "Writing to XML, if required, should be implemented externally."
src/java.management/share/classes/javax/management/modelmbean/DescriptorSupport.java
line 1067:
> 1065: }
> 1066:
> 1067: @SuppressWarnings("removal")
Can we add a comment to indicate that XMLParseException is deprecated for
removal?
src/java.management/share/classes/javax/management/modelmbean/DescriptorSupport.java
line 1131:
> 1129: * without the parentheses.
> 1130: */
> 1131: @SuppressWarnings("removal")
Can we add a comment to indicate that XMLParseException is deprecated for
removal?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23038#discussion_r1951205306
PR Review Comment: https://git.openjdk.org/jdk/pull/23038#discussion_r1951207163
PR Review Comment: https://git.openjdk.org/jdk/pull/23038#discussion_r1951195117
PR Review Comment: https://git.openjdk.org/jdk/pull/23038#discussion_r1951200677
