8138704: CertStatusReqItemV2 should not implement StatusRequest interface
Summary: Sets class to have proper inheritance and reduces visibility of length and send methods.
Reviewed-by: mullan
--- a/jdk/src/java.base/share/classes/sun/security/ssl/CertStatusReqItemV2.java Fri Oct 02 13:21:46 2015 +0900
+++ b/jdk/src/java.base/share/classes/sun/security/ssl/CertStatusReqItemV2.java Fri Oct 02 11:14:31 2015 -0700
@@ -49,7 +49,7 @@
* enum { ocsp(1), ocsp_multi(2), (255) } CertificateStatusType;
*/
-final class CertStatusReqItemV2 implements StatusRequest {
+final class CertStatusReqItemV2 {
private final StatusRequestType statReqType;
private final StatusRequest request;
@@ -144,8 +144,7 @@
*
* @return the encoded length of this {@code CertStatusReqItemV2}
*/
- @Override
- public int length() {
+ int length() {
// The length is the the status type (1 byte) + the request length
// field (2 bytes) + the StatusRequest data length.
return request.length() + 3;
@@ -159,8 +158,7 @@
*
* @throws IOException if any errors occur during the encoding process
*/
- @Override
- public void send(HandshakeOutStream s) throws IOException {
+ void send(HandshakeOutStream s) throws IOException {
s.putInt8(statReqType.id);
s.putInt16(request.length());
request.send(s);