On Wed, 22 Jan 2025 09:23:10 GMT, Hannes Wallnöfer <[email protected]> wrote:
>> src/java.datatransfer/share/classes/java/awt/datatransfer/DataFlavor.java
>> line 1288:
>>
>>> 1286:
>>> 1287: /**
>>> 1288: * Serializes this {@code DataFlavor}.
>>
>> This most definitely changes the serialisation spec. So a CSR is needed.
>> Also shouldn't readExternal be updated to correspond ?
>
> Only the `writeExternal` method is required to have a `@serialData` tag in
> order to keep javadoc running with `-serialwarn` option from complaining. I
> guess the thinking is that the `readExternal` logic can be derived from that.
>
> @prrace do you have any suggestions for the spec change, or are you ok with
> the proposed wording?
There's not a great number of "good" examples of this in the JDK, so probably
OK except it
seems like most cases will do it like a normal javadoc method so you'd want an
@param tag too.
note : this is a desktop class, so I'm looking at this one but someone else
will need to look at all the others
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22980#discussion_r1925928563
