Re: RFR: 8286395: Address possibly lossy conversions in java.security.jgss

2022-06-27 Thread Bradford Wetmore
On Fri, 17 Jun 2022 16:09:41 GMT, Ryan Ernst wrote: > Applied required casts in java.security.jgss for the upcoming warning. > Verified by cherry-picking @asotona's patch. Security didn't get a chance to review (was out last week), but looks ok to me. - PR: https://git.openjdk.org/

Re: RFR: 8286395: Address possibly lossy conversions in java.security.jgss

2022-06-21 Thread Daniel Fuchs
On Fri, 17 Jun 2022 16:09:41 GMT, Ryan Ernst wrote: > Applied required casts in java.security.jgss for the upcoming warning. > Verified by cherry-picking @asotona's patch. Looks reasonable to me - but it we be good to have someone from security-dev take a look. - PR: https://git.o

Re: RFR: 8286395: Address possibly lossy conversions in java.security.jgss

2022-06-17 Thread kristylee88
On Fri, 17 Jun 2022 16:09:41 GMT, Ryan Ernst wrote: > Applied required casts in java.security.jgss for the upcoming warning. > Verified by cherry-picking @asotona's patch. Marked as reviewed by kristyle...@github.com (no known OpenJDK username). - PR: https://git.openjdk.org/jdk/pu

Re: RFR: 8286395: Address possibly lossy conversions in java.security.jgss

2022-06-17 Thread Chris Hegarty
On Fri, 17 Jun 2022 16:09:41 GMT, Ryan Ernst wrote: > Applied required casts in java.security.jgss for the upcoming warning. > Verified by cherry-picking @asotona's patch. Looks good to me. I'm happy to sponsor this change, once a reviewer from the security area has had a chance to take a look.