infernap12/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2026-02-27 18:50:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

8138704: CertStatusReqItemV2 should not implement StatusRequest interface

Browse Source 
Sets class to have proper inheritance and reduces visibility of length and send methods.

Reviewed-by: mullan
This commit is contained in:
Jamil Nimeh 2015-10-02 11:14:31 -07:00
parent ff7c1c3bca
commit 3ef293dbc1
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
jdk/src/java.base/share/classes/sun/security/ssl/CertStatusReqItemV2.java
View File

@ -49,7 +49,7 @@ import javax.net.ssl.SSLException;
* 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 @@ final class CertStatusReqItemV2 implements StatusRequest {
*
* @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 @@ final class CertStatusReqItemV2 implements StatusRequest {
*
* @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);