RFR: 8231622: SuppressWarning("serial") ignored on field serialVersionUID [v4]

[Jonathan Gibbons]

On Wed, 9 Dec 2020 08:04:46 GMT, Guoxiong Li <github.com+13688759+lgxbslgx at openjdk.org> wrote:

>> Hi all,
>> 
>> Currently, a warning is generated when compiling the following code by using the option `-Xlint`.
>> 
>> import java.io.Serializable;
>> class B implements Serializable {
>>     @SuppressWarnings("serial")
>>     private static final byte serialVersionUID = 5;
>> }
>> 
>> output:
>> B.java:4: warning: [serial] serialVersionUID must be of type long in class B
>>     private static final byte serialVersionUID = 5;
>>                               ^
>> 1 warning
>> 
>> The annotation `@SuppressWarnings("serial")` on  field serialVersionUID is ignored by the compiler.
>> This patch fixes it and adds some tests.
>> Thank you for taking the time to review.
>> 
>> Best Regards.
>
> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use method ToolBox.checkEqual

Latest change looks good

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

Marked as reviewed by jjg (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1626