In the StatusResponseManager.get() method spec, the SSLHandshakeException is 
declared as throws exception. However, no such checked-exception would be 
thrown in the method implementation. 

/** 
... 
 * @throws SSLHandshakeException if an unsupported 
 * {@code CertStatusRequest} is provided. 
 */ 
Map<X509Certificate, byte[]> get(CertStatusRequestType type, 
            CertStatusRequest request, X509Certificate[] chain, long delay, 
            TimeUnit unit) {

As the exception is a checked-exception, and is not declared in the method, it 
is safe to remove the throws spec.

Cleanup only, no new regression test.

Bug: https://bugs.openjdk.java.net/browse/JDK-8257725

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

Commit messages:
 - 8257725: No throws of SSLHandshakeException
 - 8257724: Incorrect package of the linked class in BaseSSLSocketImpl

Changes: https://git.openjdk.java.net/jdk/pull/1607/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1607&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8257725
  Stats: 7 lines in 2 files changed: 1 ins; 3 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1607.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1607/head:pull/1607

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

Reply via email to