On Tue, 22 Dec 2020 17:16:15 GMT, Xue-Lei Andrew Fan <[email protected]> wrote:

> The local variable "knownSignatureSchemes" in the 
> CRSignatureSchemesConsumer.consume() method is assigned, but it is not really 
> queried. It is safe to remove the local variable and the related code.
> 
> Code cleanup, no new regression test.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8258828

Looks ok as long as you don't need this in chc.peerRequestedSignatureSchemes.

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

Marked as reviewed by wetmore (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1867

Reply via email to