Re: [PATCH 1/2] Remove nsdb_display_fedfsconnectionsec()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 02/08/2018 12:13 PM, Chuck Lever wrote:
> This is a display helper function that is not needed in nfs-utils
> because nfs-utils does not implement the FedFS ADMIN protocol (and
> therefore does not need to display ADMIN protocol-related security
> settings).
> 
> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx>
Committed... 

steved.

> ---
>  support/include/junction.h |    1 -
>  support/junction/display.c |   20 --------------------
>  2 files changed, 21 deletions(-)
> 
> diff --git a/support/include/junction.h b/support/include/junction.h
> index a5f3844..67cf017 100644
> --- a/support/include/junction.h
> +++ b/support/include/junction.h
> @@ -118,7 +118,6 @@ FedFsStatus	 nsdb_posix_to_path_array(const char *pathname,
>   ** Readability helpers
>   **/
>  
> -const char      *nsdb_display_fedfsconnectionsec(const FedFsConnectionSec sectype);
>  const char      *nsdb_display_fedfsstatus(const FedFsStatus status);
>  void             nsdb_print_fedfsstatus(const FedFsStatus status);
>  
> diff --git a/support/junction/display.c b/support/junction/display.c
> index 77b131f..e1e1af1 100644
> --- a/support/junction/display.c
> +++ b/support/junction/display.c
> @@ -32,26 +32,6 @@
>  #include "junction.h"
>  
>  /**
> - * Return human-readable equivalent of a FedFsConnectionSec value
> - *
> - * @param sectype FedFsConneccionSec value
> - * @return a static NUL-terminated C string
> - */
> -const char *
> -nsdb_display_fedfsconnectionsec(const FedFsConnectionSec sectype)
> -{
> -	switch (sectype) {
> -	case FEDFS_SEC_NONE:
> -		return "FEDFS_SEC_NONE";
> -	case FEDFS_SEC_TLS:
> -		return "FEDFS_SEC_TLS";
> -	default:
> -		break;
> -	}
> -	return "unrecognized security type";
> -}
> -
> -/**
>   * Return human-readable equivalent of a FedFsStatus value
>   *
>   * @param status FedFsStatus code
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux