[FFmpeg-devel] [PATCH 3/7] libavfilter: Fixes warnings for unused variables in dnn_srcnn.h, dnn_espcn.h, dnn_backend_tf.c.

[Pedro Arthur]

2018-08-02 15:52 GMT-03:00 Sergey Lavrushkin <dualfal at gmail.com>:
> ---
>  libavfilter/dnn_backend_tf.c | 64 +++++++++++++++++++++++++++++++++++++++++++-
>  libavfilter/dnn_espcn.h      | 37 -------------------------
>  libavfilter/dnn_srcnn.h      | 35 ------------------------
>  3 files changed, 63 insertions(+), 73 deletions(-)
>
> diff --git a/libavfilter/dnn_backend_tf.c b/libavfilter/dnn_backend_tf.c
> index 6307c794a5..7a4ad72d27 100644
> --- a/libavfilter/dnn_backend_tf.c
> +++ b/libavfilter/dnn_backend_tf.c
> @@ -374,9 +374,71 @@ DNNModel *ff_dnn_load_default_model_tf(DNNDefaultModel model_type)
>      TFModel *tf_model = NULL;
>      TF_OperationDescription *op_desc;
>      TF_Operation *op;
> -    TF_Operation *const_ops_buffer[6];
>      TF_Output input;
>      int64_t input_shape[] = {1, -1, -1, 1};
> +    const char tanh[] = "Tanh";
> +    const char sigmoid[] = "Sigmoid";
> +    const char relu[] = "Relu";
> +
> +    const float *srcnn_consts[] = {
> +        srcnn_conv1_kernel,
> +        srcnn_conv1_bias,
> +        srcnn_conv2_kernel,
> +        srcnn_conv2_bias,
> +        srcnn_conv3_kernel,
> +        srcnn_conv3_bias
> +    };
> +    const long int *srcnn_consts_dims[] = {
> +        srcnn_conv1_kernel_dims,
> +        srcnn_conv1_bias_dims,
> +        srcnn_conv2_kernel_dims,
> +        srcnn_conv2_bias_dims,
> +        srcnn_conv3_kernel_dims,
> +        srcnn_conv3_bias_dims
> +    };
> +    const int srcnn_consts_dims_len[] = {
> +        4,
> +        1,
> +        4,
> +        1,
> +        4,
> +        1
> +    };
> +    const char *srcnn_activations[] = {
> +        relu,
> +        relu,
> +        relu
> +    };
> +
> +    const float *espcn_consts[] = {
> +        espcn_conv1_kernel,
> +        espcn_conv1_bias,
> +        espcn_conv2_kernel,
> +        espcn_conv2_bias,
> +        espcn_conv3_kernel,
> +        espcn_conv3_bias
> +    };
> +    const long int *espcn_consts_dims[] = {
> +        espcn_conv1_kernel_dims,
> +        espcn_conv1_bias_dims,
> +        espcn_conv2_kernel_dims,
> +        espcn_conv2_bias_dims,
> +        espcn_conv3_kernel_dims,
> +        espcn_conv3_bias_dims
> +    };
> +    const int espcn_consts_dims_len[] = {
> +        4,
> +        1,
> +        4,
> +        1,
> +        4,
> +        1
> +    };
> +    const char *espcn_activations[] = {
> +        tanh,
> +        tanh,
> +        sigmoid
> +    };
>
>      input.index = 0;
>
> diff --git a/libavfilter/dnn_espcn.h b/libavfilter/dnn_espcn.h
> index a0dd61cd0d..9344aa90fe 100644
> --- a/libavfilter/dnn_espcn.h
> +++ b/libavfilter/dnn_espcn.h
> @@ -5398,41 +5398,4 @@ static const long int espcn_conv3_bias_dims[] = {
>      4
>  };
>
> -static const float *espcn_consts[] = {
> -    espcn_conv1_kernel,
> -    espcn_conv1_bias,
> -    espcn_conv2_kernel,
> -    espcn_conv2_bias,
> -    espcn_conv3_kernel,
> -    espcn_conv3_bias
> -};
> -
> -static const long int *espcn_consts_dims[] = {
> -    espcn_conv1_kernel_dims,
> -    espcn_conv1_bias_dims,
> -    espcn_conv2_kernel_dims,
> -    espcn_conv2_bias_dims,
> -    espcn_conv3_kernel_dims,
> -    espcn_conv3_bias_dims
> -};
> -
> -static const int espcn_consts_dims_len[] = {
> -    4,
> -    1,
> -    4,
> -    1,
> -    4,
> -    1
> -};
> -
> -static const char espcn_tanh[] = "Tanh";
> -
> -static const char espcn_sigmoid[] = "Sigmoid";
> -
> -static const char *espcn_activations[] = {
> -    espcn_tanh,
> -    espcn_tanh,
> -    espcn_sigmoid
> -};
> -
>  #endif
> diff --git a/libavfilter/dnn_srcnn.h b/libavfilter/dnn_srcnn.h
> index 26143654b8..4f5332ce18 100644
> --- a/libavfilter/dnn_srcnn.h
> +++ b/libavfilter/dnn_srcnn.h
> @@ -2110,39 +2110,4 @@ static const long int srcnn_conv3_bias_dims[] = {
>      1
>  };
>
> -static const float *srcnn_consts[] = {
> -    srcnn_conv1_kernel,
> -    srcnn_conv1_bias,
> -    srcnn_conv2_kernel,
> -    srcnn_conv2_bias,
> -    srcnn_conv3_kernel,
> -    srcnn_conv3_bias
> -};
> -
> -static const long int *srcnn_consts_dims[] = {
> -    srcnn_conv1_kernel_dims,
> -    srcnn_conv1_bias_dims,
> -    srcnn_conv2_kernel_dims,
> -    srcnn_conv2_bias_dims,
> -    srcnn_conv3_kernel_dims,
> -    srcnn_conv3_bias_dims
> -};
> -
> -static const int srcnn_consts_dims_len[] = {
> -    4,
> -    1,
> -    4,
> -    1,
> -    4,
> -    1
> -};
> -
> -static const char srcnn_relu[] = "Relu";
> -
> -static const char *srcnn_activations[] = {
> -    srcnn_relu,
> -    srcnn_relu,
> -    srcnn_relu
> -};
> -
>  #endif
> --
> 2.14.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


LGTM.
I'll push it by tomorrow.