Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v3]

2023-01-03 Thread Ryan Wallace
On Fri, 25 Nov 2022 06:08:30 GMT, Xue-Lei Andrew Fan wrote: >> Ryan Wallace has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 14 additional >> com

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v4]

2023-01-03 Thread Ryan Wallace
> SubjectCodeSource is no longer used, Class should be deleted and all > references updated Ryan Wallace has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contain

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v3]

2022-11-24 Thread Xue-Lei Andrew Fan
On Thu, 24 Nov 2022 09:02:51 GMT, Ryan Wallace wrote: >> SubjectCodeSource is no longer used, Class should be deleted and all >> references updated > > Ryan Wallace has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated ch

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-24 Thread Ryan Wallace
On Fri, 18 Nov 2022 17:49:45 GMT, Xue-Lei Andrew Fan wrote: >> @XueleiFan I have removed the Test and related policy files >> (Comparator.Combined.Policy/ >> Comparator.Comparator.Policy/Comparator.Principal.Policy), is this what you >> were expecting? I can confirm I am still getting a success

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v3]

2022-11-24 Thread Ryan Wallace
> SubjectCodeSource is no longer used, Class should be deleted and all > references updated Ryan Wallace has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contain

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-18 Thread Xue-Lei Andrew Fan
On Fri, 18 Nov 2022 13:07:06 GMT, Ryan Wallace wrote: >> test/jdk/sun/security/provider/PolicyFile/Comparator.java line 36: >> >>> 34: * >>> 35: * Note: SubjectCodeSource removed, but PolicyFile might have the same >>> problem. Keep the test as-is >>> 36: * >> >> I may have the note in a bu

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-18 Thread Ryan Wallace
On Tue, 8 Nov 2022 16:41:07 GMT, Xue-Lei Andrew Fan wrote: >> Ryan Wallace has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains seven additional >> c

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-10 Thread Sean Mullan
On Thu, 10 Nov 2022 15:33:26 GMT, Sean Mullan wrote: >> Ryan Wallace has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains seven additional >> commits

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-10 Thread Ryan Wallace
On Thu, 10 Nov 2022 15:33:26 GMT, Sean Mullan wrote: >> Ryan Wallace has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains seven additional >> commits

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-10 Thread Sean Mullan
On Tue, 8 Nov 2022 10:01:47 GMT, Ryan Wallace wrote: >> SubjectCodeSource is no longer used, Class should be deleted and all >> references updated > > Ryan Wallace has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated cha

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-08 Thread Xue-Lei Andrew Fan
On Tue, 8 Nov 2022 10:01:47 GMT, Ryan Wallace wrote: >> SubjectCodeSource is no longer used, Class should be deleted and all >> references updated > > Ryan Wallace has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated cha

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced [v2]

2022-11-08 Thread Ryan Wallace
> SubjectCodeSource is no longer used, Class should be deleted and all > references updated Ryan Wallace has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contain

Re: RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced

2022-11-08 Thread Weijun Wang
On Tue, 4 Oct 2022 08:25:10 GMT, Ryan Wallace wrote: > SubjectCodeSource is no longer used, Class should be deleted and all > references updated Marked as reviewed by weijun (Reviewer). test/jdk/sun/security/provider/PolicyFile/Comparator.java line 27: > 25: * @test > 26: * @bug 5037004 > 2

RFR: 8294526: sun/security/provider/SubjectCodeSource.java no longer referenced

2022-11-08 Thread Ryan Wallace
SubjectCodeSource is no longer used, Class should be deleted and all references updated - Commit messages: - Merge branch 'master' into 8294526 - Merge branch 'master' into 8294526 - 8294526: SubjectCodeSource is no longer referenced - Merge branch 'master' into 8294526 - Merge