Drop DEFINE_STACK_OF.

We never need to define the actual structs because we always cast them
before use. The types only exist to be distinct, and they can do that
without a definition.

Change-Id: I1e1ca0833b383f3be422675cb7b90dacbaf82acf
Reviewed-on: https://boringssl-review.googlesource.com/5593
Reviewed-by: David Benjamin <davidben@chromium.org>
Reviewed-by: Adam Langley <agl@google.com>

include/openssl/asn1.h

@@ -515,7 +515,6 @@ struct X509_algor_st
        ASN1_OBJECT *algorithm;
        ASN1_TYPE *parameter;
        } /* X509_ALGOR */;
-DEFINE_STACK_OF(X509_ALGOR);
 
 DECLARE_ASN1_FUNCTIONS(X509_ALGOR)
 

include/openssl/bio.h

@@ -76,8 +76,6 @@ extern "C" {
 
 /* Allocation and freeing. */
 
-DEFINE_STACK_OF(BIO);
-
 /* BIO_new creates a new BIO with the given type and a reference count of one.
  * It returns the fresh |BIO|, or NULL on error. */
 OPENSSL_EXPORT BIO *BIO_new(const BIO_METHOD *type);

include/openssl/stack.h

@@ -111,11 +111,6 @@ typedef struct stack_st {
 
 #define STACK_OF(type) struct stack_st_##type
 
-#define DEFINE_STACK_OF(type) \
-STACK_OF(type) {\
-  _STACK stack; \
-}
-
 #define DECLARE_STACK_OF(type) STACK_OF(type);
 
 /* The make_macros.sh script in this directory parses the following lines and