RFR: 8178806: Better exception logging in crypto code [v2]

Kevin Driver kdriver at openjdk.org
Thu Feb 9 22:15:08 UTC 2023


On Thu, 9 Feb 2023 21:47:17 GMT, Kevin Driver <kdriver at openjdk.org> wrote:

>> src/java.base/share/classes/javax/crypto/JceSecurity.java.template line 240:
>> 
>>> 238:         Exception e = getVerificationResult(p);
>>> 239:         if (debug != null) {
>>> 240:             if(e != null) {
>> 
>> line 239 and 240 can be combined into one line?
>
> I thought about this, but logically they are not related... 
> 
> I can definitely combine, but I made this choice for readability. What do you think?

resolved

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

PR: https://git.openjdk.org/jdk/pull/12504



More information about the security-dev mailing list