On Mon, 10 Oct 2022 21:04:46 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> Mark Powers has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Sean comments
>
> test/jdk/javax/security/auth/PrivateCredentialPermission/Serial2.java line 35:
>
>> 33: import java.util.*;
>> 34:
>> 35: public class Serial2 implements java.io.Serializable {
>
> I don't see why this needs to implement `Serializable` (I see you probably
> used the other `Serial.java` test in this directory as a template. I don't
> think that needs it either, but you can leave that as is for now).
I read your sentence as "leave both for now". Sorry. You mean fix in this PR
and leave the other alone.
-------------
PR: https://git.openjdk.org/jdk/pull/10206
