[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