Remove allow dead code from param builder

Author: swenson

openssl/src/ossl_param.rs

@@ -9,10 +9,6 @@
 //! Note, that this module is available only in OpenSSL 3.* and
 //! only internally for this crate!
 
-// Depending on which version of OpenSSL is used, and which algorithms
-// are exposed in the bindings, not all of these functions are used.
-#![allow(dead_code)]
-
 use crate::bn::{BigNum, BigNumRef};
 use crate::error::ErrorStack;
 use crate::util;
@@ -36,6 +32,7 @@ foreign_type_and_impl_send_sync! {
 impl OsslParamRef {
     /// Locates the `OsslParam` in the `OsslParam` array
     #[corresponds(OSSL_PARAM_locate)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn locate(&self, key: &CStr) -> Result<&OsslParamRef, ErrorStack> {
         unsafe {
             let param = cvt_p(ffi::OSSL_PARAM_locate(self.as_ptr(), key.as_ptr()))?;
@@ -45,6 +42,7 @@ impl OsslParamRef {
 
     /// Get `BigNum` from the current `OsslParam`
     #[corresponds(OSSL_PARAM_get_BN)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn get_bn(&self) -> Result<BigNum, ErrorStack> {
         unsafe {
             let mut bn: *mut ffi::BIGNUM = ptr::null_mut();
@@ -55,6 +53,7 @@ impl OsslParamRef {
 
     /// Get `&str` from the current `OsslParam`
     #[corresponds(OSSL_PARAM_get_utf8_string)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn get_utf8_string(&self) -> Result<&str, ErrorStack> {
         unsafe {
             let mut val: *const c_char = ptr::null_mut();
@@ -65,6 +64,7 @@ impl OsslParamRef {
 
     /// Get octet string (as `&[u8]) from the current `OsslParam`
     #[corresponds(OSSL_PARAM_get_octet_string)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn get_octet_string(&self) -> Result<&[u8], ErrorStack> {
         unsafe {
             let mut val: *const c_void = ptr::null_mut();
@@ -94,6 +94,7 @@ impl OsslParamBuilder {
     ///
     /// The array is initially empty.
     #[corresponds(OSSL_PARAM_BLD_new)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn new() -> Result<OsslParamBuilder, ErrorStack> {
         unsafe {
             ffi::init();
@@ -102,8 +103,10 @@ impl OsslParamBuilder {
         }
     }
 
-    /// Constructs the `OsslParam`.
+    /// Constructs the `OsslParam` and clears this builder.
     #[corresponds(OSSL_PARAM_BLD_to_param)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
+    #[allow(clippy::wrong_self_convention)]
     pub fn to_param(&mut self) -> Result<OsslParam, ErrorStack> {
         unsafe {
             let params = cvt_p(ffi::OSSL_PARAM_BLD_to_param(self.0))?;
@@ -117,6 +120,7 @@ impl OsslParamBuilderRef {
     ///
     /// Note, that both key and bn need to exist until the `to_param` is called!
     #[corresponds(OSSL_PARAM_BLD_push_BN)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn add_bn(&mut self, key: &CStr, bn: &BigNumRef) -> Result<(), ErrorStack> {
         unsafe {
             cvt(ffi::OSSL_PARAM_BLD_push_BN(
@@ -132,6 +136,7 @@ impl OsslParamBuilderRef {
     ///
     /// Note, that both `key` and `buf` need to exist until the `to_param` is called!
     #[corresponds(OSSL_PARAM_BLD_push_utf8_string)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn add_utf8_string(&mut self, key: &CStr, buf: &str) -> Result<(), ErrorStack> {
         unsafe {
             cvt(ffi::OSSL_PARAM_BLD_push_utf8_string(
@@ -148,6 +153,7 @@ impl OsslParamBuilderRef {
     ///
     /// Note, that both `key` and `buf` need to exist until the `to_param` is called!
     #[corresponds(OSSL_PARAM_BLD_push_octet_string)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn add_octet_string(&mut self, key: &CStr, buf: &[u8]) -> Result<(), ErrorStack> {
         unsafe {
             cvt(ffi::OSSL_PARAM_BLD_push_octet_string(
@@ -164,6 +170,7 @@ impl OsslParamBuilderRef {
     ///
     /// Note, that both `key` and `buf` need to exist until the `to_param` is called!
     #[corresponds(OSSL_PARAM_BLD_push_uint)]
+    #[cfg_attr(any(not(ossl320), osslconf = "OPENSSL_NO_ARGON2"), allow(dead_code))]
     pub fn add_uint(&mut self, key: &CStr, val: u32) -> Result<(), ErrorStack> {
         unsafe {
             cvt(ffi::OSSL_PARAM_BLD_push_uint(