Skip to content

Commit

Permalink
Remove unused internal constant SSL::ExtConfig
Browse files Browse the repository at this point in the history
  • Loading branch information
kateinoigakukun committed Dec 22, 2022
1 parent 7a9b09e commit ad098d0
Showing 1 changed file with 1 addition and 18 deletions.
19 changes: 1 addition & 18 deletions ext/openssl/ossl_ssl.c
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@
TypedData_Get_Struct((obj), SSL_CTX, &ossl_sslctx_type, (ctx)); \
} while (0)

static VALUE mSSLExtConfig;
#ifndef OPENSSL_NO_SOCK

VALUE mSSL;
Expand Down Expand Up @@ -2591,21 +2590,7 @@ Init_ossl_ssl(void)
mSSL = rb_define_module_under(mOSSL, "SSL");
#endif /* !defined(OPENSSL_NO_SOCK) */

/* Document-module: OpenSSL::ExtConfig
*
* This module contains configuration information about the SSL extension,
* for example if socket support is enabled, or the host name TLS extension
* is enabled. Constants in this module will always be defined, but contain
* +true+ or +false+ values depending on the configuration of your OpenSSL
* installation.
*/
mSSLExtConfig = rb_define_module_under(mOSSL, "ExtConfig");

#ifdef OPENSSL_NO_SOCK
rb_define_const(mSSLExtConfig, "OPENSSL_NO_SOCK", Qfalse);
#else
rb_define_const(mSSLExtConfig, "OPENSSL_NO_SOCK", Qtrue);

#ifndef OPENSSL_NO_SOCK
/* Document-class: OpenSSL::SSL::SSLError
*
* Generic error class raised by SSLSocket and SSLContext.
Expand Down Expand Up @@ -2768,8 +2753,6 @@ Init_ossl_ssl(void)
*/
rb_attr(cSSLContext, rb_intern_const("session_remove_cb"), 1, 1, Qfalse);

rb_define_const(mSSLExtConfig, "HAVE_TLSEXT_HOST_NAME", Qtrue);

This comment has been minimized.

Copy link
@rhenium

rhenium Dec 22, 2022

Member

Good catch. This also seems unnecessary with commit 4eb4b32.

/*
* A callback invoked whenever a new handshake is initiated on an
* established connection. May be used to disable renegotiation entirely.
Expand Down

0 comments on commit ad098d0

Please sign in to comment.