RFR: 8347433: Deprecate XML interchange in java.management/javax/management/modelmbean/DescriptorSupport for removal [v2]

[Daniel Fuchs]

On Fri, 21 Feb 2025 06:36:14 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

>> Kevin Walls has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains eight additional commits since the last revision:
>> 
>>  - comments
>>  - Merge remote-tracking branch 'upstream/master' into 8347433_DescriptorSupport_deprecate_XML
>>  - (c)
>>  - typo
>>  - Merge remote-tracking branch 'upstream/master' into 8347433_DescriptorSupport_deprecate_XML
>>  - Also XMLParseException
>>  - RMMB comment update
>>  - Deprecate XML interchange in DescriptorSupport
>
> src/java.management/share/classes/javax/management/modelmbean/DescriptorSupport.java line 1170:
> 
>> 1168:                 " with arg: <" + s + ">";
>> 1169:             throw new XMLParseException(e, msg);
>> 1170:             // XMLParseException is deprecated for removal.
> 
> Q: Non-deprecated method will be using the deprecated class `XMLParseException`. What is the plan for replacement? Should this be fixed now? The same question for the line 1160.

Aside from the public deprecated API, the only places where this exception is used is in private static methods that support the implementation of the deprecated constructor, so these will go away when the constructor is eventually removed.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23038#discussion_r1965157527