POD: stop abusing comment
authorRichard Levitte <levitte@openssl.org>
Sun, 29 Sep 2019 15:10:59 +0000 (17:10 +0200)
committerRichard Levitte <levitte@openssl.org>
Fri, 11 Oct 2019 13:30:57 +0000 (15:30 +0200)
OpenSSL uses some POD directives masquerading as 'comment'
('=for comment' etc).  This is abusive and confusing.  Instead, we use
our own keyword.

    =for openssl whatever

    =begin openssl

    whatever

    =end openssl

(we have never used the multiline form, but might start one day)

Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/10048)

35 files changed:
doc/internal/man3/DEFINE_SPARSE_ARRAY_OF.pod
doc/internal/man3/ossl_param_bld_init.pod
doc/man1/openssl-cmds.pod
doc/man1/openssl-ts.pod
doc/man1/tsget.pod
doc/man3/BIO_f_base64.pod
doc/man3/BIO_f_cipher.pod
doc/man3/BIO_f_md.pod
doc/man3/BIO_f_ssl.pod
doc/man3/BIO_get_ex_new_index.pod
doc/man3/DEFINE_STACK_OF.pod
doc/man3/EVP_EncryptInit.pod
doc/man3/EVP_aes.pod
doc/man3/EVP_aria.pod
doc/man3/EVP_camellia.pod
doc/man3/EVP_des.pod
doc/man3/OPENSSL_LH_COMPFUNC.pod
doc/man3/OSSL_PARAM_int.pod
doc/man3/OSSL_trace_enabled.pod
doc/man3/OSSL_trace_set_channel.pod
doc/man3/PEM_read_CMS.pod
doc/man3/SSL_get_all_async_fds.pod
doc/man3/SSL_set_async_callback.pod
doc/man3/X509_dup.pod
doc/man3/d2i_X509.pod
doc/man7/bio.pod
doc/man7/ossl_store-file.pod
doc/man7/ossl_store.pod
doc/man7/provider-cipher.pod
doc/man7/provider-digest.pod
doc/man7/provider-keyexch.pod
doc/man7/provider-mac.pod
doc/man7/provider-signature.pod
doc/man7/provider.pod
util/find-doc-nits

index 7917c4c..a78193c 100644 (file)
@@ -9,7 +9,7 @@ ossl_sa_TYPE_doall_arg, ossl_sa_TYPE_get, ossl_sa_TYPE_set
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include "crypto/sparse_array.h"
 
index 5179a8d..062e2c2 100644 (file)
@@ -15,7 +15,7 @@ ossl_param_bld_push_octet_ptr
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include "internal/params_build.h"
 
index 5c4f06e..9f7d6c6 100644 (file)
@@ -55,7 +55,7 @@ x509
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
 B<openssl> I<cmd> B<-help> | [I<-option> | I<-option> I<arg>] ... [I<arg>] ...
 
index c97909d..2e095d4 100644 (file)
@@ -623,7 +623,7 @@ You could also look at the 'test' directory for more examples.
 
 =head1 BUGS
 
-=for comment foreign manuals: procmail(1), perl(1)
+=for openssl foreign manuals: procmail(1), perl(1)
 
 =over 2
 
index 19c689e..d006198 100644 (file)
@@ -185,7 +185,7 @@ example:
 
 =head1 SEE ALSO
 
-=for comment foreign manuals: WWW::Curl::Easy
+=for openssl foreign manuals: WWW::Curl::Easy
 
 L<openssl(1)>,
 L<openssl-ts(1)>,
index 6ac98e9..69de007 100644 (file)
@@ -6,7 +6,7 @@ BIO_f_base64 - base64 BIO filter
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/bio.h>
  #include <openssl/evp.h>
index 6f5dc15..c1f1ff8 100644 (file)
@@ -6,7 +6,7 @@ BIO_f_cipher, BIO_set_cipher, BIO_get_cipher_status, BIO_get_cipher_ctx - cipher
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/bio.h>
  #include <openssl/evp.h>
index cd45730..8ad6948 100644 (file)
@@ -6,7 +6,7 @@ BIO_f_md, BIO_set_md, BIO_get_md, BIO_get_md_ctx - message digest BIO filter
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/bio.h>
  #include <openssl/evp.h>
index 82bb16c..6b896e2 100644 (file)
@@ -11,7 +11,7 @@ BIO_ssl_shutdown - SSL BIO
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/bio.h>
  #include <openssl/ssl.h>
index 0d1fb5b..60f49f7 100644 (file)
@@ -17,7 +17,7 @@ RSA_get_ex_new_index, RSA_set_ex_data, RSA_get_ex_data
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/x509.h>
 
index a3e9e73..4dd3de8 100644 (file)
@@ -14,7 +14,7 @@ sk_TYPE_dup, sk_TYPE_deep_copy, sk_TYPE_set_cmp_func, sk_TYPE_new_reserve
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/safestack.h>
 
index 0f04fe0..3e66820 100644 (file)
@@ -65,7 +65,7 @@ EVP_CIPHER_do_all_ex
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/evp.h>
 
index 5a2be9a..1ca893a 100644 (file)
@@ -51,7 +51,7 @@ EVP_aes_256_xts
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/evp.h>
 
index b686edf..6471acb 100644 (file)
@@ -36,7 +36,7 @@ EVP_aria_256_gcm,
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/evp.h>
 
index 9ea3d9a..4a57f79 100644 (file)
@@ -30,7 +30,7 @@ EVP_camellia_256_ofb
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/evp.h>
 
index 1911ff6..d00179e 100644 (file)
@@ -28,7 +28,7 @@ EVP_des_ede3_wrap
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/evp.h>
 
index 804891d..412a4f8 100644 (file)
@@ -12,7 +12,7 @@ lh_TYPE_doall, lh_TYPE_doall_arg, lh_TYPE_error - dynamic hash table
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/lhash.h>
 
index 991b3d1..8d4aa90 100644 (file)
@@ -32,7 +32,7 @@ OSSL_PARAM_set_octet_ptr
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/params.h>
 
index 2ced3d4..d49a779 100644 (file)
@@ -12,7 +12,7 @@ OSSL_TRACE_ENABLED
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/trace.h>
 
index f214abd..ad06b4c 100644 (file)
@@ -1,6 +1,6 @@
 =pod
 
-=for comment foreign manuals: atexit(3)
+=for openssl foreign manuals: atexit(3)
 
 =head1 NAME
 
index a18856c..5cc3251 100644 (file)
@@ -40,7 +40,7 @@ PEM_write_bio_SSL_SESSION
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/pem.h>
 
index 5adfe90..03cd0e8 100644 (file)
@@ -9,7 +9,7 @@ SSL_get_changed_async_fds
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/async.h>
  #include <openssl/ssl.h>
index 46d2923..541b743 100644 (file)
@@ -12,7 +12,7 @@ SSL_async_callback_fn
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/ssl.h>
 
index e190100..d7d8477 100644 (file)
@@ -287,7 +287,7 @@ X509_dup,
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/asn1t.h>
 
index 12baaa9..62e6b24 100644 (file)
@@ -387,7 +387,7 @@ i2d_X509_VAL,
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  TYPE *d2i_TYPE(TYPE **a, unsigned char **ppin, long length);
  TYPE *d2i_TYPE_bio(BIO *bp, TYPE **a);
index 63380bc..bb23e56 100644 (file)
@@ -6,7 +6,7 @@ bio - Basic I/O abstraction
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
  #include <openssl/bio.h>
 
index d6ea27b..d4d163e 100644 (file)
@@ -14,7 +14,7 @@ ossl_store-file - The store 'file' scheme loader
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
 #include <openssl/store.h>
 
index 148ceef..68503cd 100644 (file)
@@ -6,7 +6,7 @@ ossl_store - Store retrieval functions
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
 #include <openssl/store.h>
 
index 2da3aa2..af97735 100644 (file)
@@ -6,7 +6,7 @@ provider-cipher - The cipher library E<lt>-E<gt> provider functions
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/core_numbers.h>
  #include <openssl/core_names.h>
index 089c0e2..7ee58c2 100644 (file)
@@ -6,7 +6,7 @@ provider-digest - The digest library E<lt>-E<gt> provider functions
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/core_numbers.h>
  #include <openssl/core_names.h>
index c4cb021..0f7752c 100644 (file)
@@ -6,7 +6,7 @@ provider-keyexch - The keyexch library E<lt>-E<gt> provider functions
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/core_numbers.h>
  #include <openssl/core_names.h>
index 70653e3..ef68374 100644 (file)
@@ -6,7 +6,7 @@ provider-mac - The mac library E<lt>-E<gt> provider functions
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/core_numbers.h>
  #include <openssl/core_names.h>
index 73d573a..fa44673 100644 (file)
@@ -6,7 +6,7 @@ provider-signature - The signature library E<lt>-E<gt> provider functions
 
 =head1 SYNOPSIS
 
-=for comment multiple includes
+=for openssl multiple includes
 
  #include <openssl/core_numbers.h>
  #include <openssl/core_names.h>
index 87a9087..f13418d 100644 (file)
@@ -6,7 +6,7 @@ provider - OpenSSL operation implementation providers
 
 =head1 SYNOPSIS
 
-=for comment generic
+=for openssl generic
 
 #include <openssl/provider.h>
 
@@ -123,8 +123,6 @@ The number for this operation is B<OSSL_OP_CIPHER>.
 The functions the provider can offer are described in
 L<provider-cipher(7)>
 
-=begin comment NOT AVAILABLE YET
-
 =item Message Authentication Code (MAC)
 
 In the OpenSSL libraries, the corresponding method object is
@@ -133,10 +131,6 @@ The number for this operation is B<OSSL_OP_MAC>.
 The functions the provider can offer are described in
 L<provider-mac(7)>
 
-=end comment
-
-=begin comment NOT AVAILABLE YET
-
 =item Key Derivation Function (KDF)
 
 In the OpenSSL libraries, the corresponding method object is
@@ -145,8 +139,6 @@ The number for this operation is B<OSSL_OP_KDF>.
 The functions the provider can offer are described in
 L<provider-kdf(7)>
 
-=end comment
-
 =item Key Exchange
 
 In the OpenSSL libraries, the corresponding method object is
index eac87de..30cdd77 100755 (executable)
@@ -444,7 +444,7 @@ sub check {
         check_section_location($id, $contents, "EXAMPLES", "SEE ALSO");
     }
 
-    unless ( $contents =~ /=for comment generic/ ) {
+    unless ( $contents =~ /=for openssl generic/ ) {
         if ( $filename =~ m|man3/| ) {
             name_synopsis($id, $filename, $contents);
             functionname_check($id, $filename, $contents);
@@ -482,7 +482,7 @@ sub check {
     err($id, "Possible version style issue")
         if $contents =~ /OpenSSL version [019]/;
 
-    if ( $contents !~ /=for comment multiple includes/ ) {
+    if ( $contents !~ /=for openssl multiple includes/ ) {
         # Look for multiple consecutive openssl #include lines
         # (non-consecutive lines are okay; see man3/MD5.pod).
         if ( $contents =~ /=head1 SYNOPSIS(.*)=head1 DESCRIPTION/ms ) {