On Thu, 3 Aug 2023 22:49:46 GMT, Valerie Peng <valer...@openjdk.org> wrote:

>> Pavel Rappo has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Feedback
>
> src/java.base/share/classes/com/sun/crypto/provider/PBKDF2KeyImpl.java line 
> 272:
> 
>> 270:             }
>> 271: 
>> 272:             if 
>> (!(that.getAlgorithm().equalsIgnoreCase(getAlgorithm()))){
> 
> why removing the space before the "{"?

It surprises me as much as it probably surprises you. Looks like a sloppy merge 
(821f617ffae) on my part.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14738#discussion_r1284208096

Reply via email to