RFR: JDK-8291974 PrivateCredentialPermission should not use local variable to enable debugging [v4]

Mark Powers mpowers at openjdk.org
Mon Oct 31 15:02:30 UTC 2022


On Mon, 10 Oct 2022 21:04:46 GMT, Sean Mullan <mullan at 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



More information about the security-dev mailing list