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

[Sean Mullan]

On 10/31/22 11:02 AM, Mark Powers wrote:
> 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.

Yes, I meant remove it from this test if it isn't needed. I think you 
can fix the other one, if you like, but since it hasn't caused any 
issues, feel free to leave it for now.

--Sean