[Spice-devel] [PATCH 14/30] More encapsulation for dcc_encoders_free

Jonathon Jongsma jjongsma at redhat.com
Wed Jun 8 21:56:17 UTC 2016


On Tue, 2016-06-07 at 11:17 +0100, Frediano Ziglio wrote:
> Encoders function should not use DisplayChannelClient
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  server/dcc-encoders.c | 3 +--
>  server/dcc-encoders.h | 2 +-
>  server/dcc.c          | 2 +-
>  3 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/server/dcc-encoders.c b/server/dcc-encoders.c
> index a6e7b2c..e606ded 100644
> --- a/server/dcc-encoders.c
> +++ b/server/dcc-encoders.c
> @@ -415,9 +415,8 @@ void dcc_encoders_init(DisplayChannelClient *dcc)
>      enc->zlib_level = ZLIB_DEFAULT_COMPRESSION_LEVEL;
>  }
>  
> -void dcc_encoders_free(DisplayChannelClient *dcc)
> +void dcc_encoders_free(EncodersData *enc)

Change to encoders_data_free()?

>  {
> -    EncodersData *enc = &dcc->encoders;
>      quic_destroy(enc->quic);
>      enc->quic = NULL;
>      lz_destroy(enc->lz);
> diff --git a/server/dcc-encoders.h b/server/dcc-encoders.h
> index f1c4edf..b9c81e8 100644
> --- a/server/dcc-encoders.h
> +++ b/server/dcc-encoders.h
> @@ -37,7 +37,7 @@ typedef struct RedGlzDrawable RedGlzDrawable;
>  typedef struct EncodersData EncodersData;
>  
>  void             dcc_encoders_init                           (DisplayChannelC
> lient *dcc);
> -void             dcc_encoders_free                           (DisplayChannelC
> lient *dcc);
> +void dcc_encoders_free(EncodersData *enc);
>  void             dcc_free_glz_drawable                       (DisplayChannelC
> lient *dcc,
>                                                                RedGlzDrawable
> *drawable);
>  int              dcc_free_some_independent_glz_drawables     (DisplayChannelC
> lient *dcc,
> diff --git a/server/dcc.c b/server/dcc.c
> index 46182f9..40a465e 100644
> --- a/server/dcc.c
> +++ b/server/dcc.c
> @@ -499,7 +499,7 @@ void dcc_stop(DisplayChannelClient *dcc)
>      free(dcc->send_data.stream_outbuf);
>      free(dcc->send_data.free_list.res);
>      dcc_destroy_stream_agents(dcc);
> -    dcc_encoders_free(dcc);
> +    dcc_encoders_free(&dcc->encoders);
>  
>      if (dcc->gl_draw_ongoing) {
>          display_channel_gl_draw_done(dc);

Reviewed-by: Jonathon Jongsma <jjongsma at redhat.com>


More information about the Spice-devel mailing list