RFR: 8258828: The method local variable is not really used

Bradford Wetmore wetmore at openjdk.java.net
Tue Dec 22 18:13:57 UTC 2020


On Tue, 22 Dec 2020 17:16:15 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> 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



More information about the security-dev mailing list