Re: [PATCH] CLEANUP: compression: remove unused deinit_comp_ctx section

2020-01-15 Thread William Lallemand 
On Sat, Jan 11, 2020 at 11:40:42AM +0100, William Dauchy wrote:
> Since commit 27d93c3f9428 ("BUG/MAJOR: compression/cache: Make it
> really works with these both filters"), we no longer use section
> deinit_comp_ctx.
> 
> This should fix github issue #441
> 

Thanks, applied!

-- 
William Lallemand

[PATCH] CLEANUP: compression: remove unused deinit_comp_ctx section

2020-01-11 Thread William Dauchy 
Since commit 27d93c3f9428 ("BUG/MAJOR: compression/cache: Make it
really works with these both filters"), we no longer use section
deinit_comp_ctx.

This should fix github issue #441

Signed-off-by: William Dauchy 
---
 src/flt_http_comp.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/flt_http_comp.c b/src/flt_http_comp.c
index e98ff46b..574b9373 100644
--- a/src/flt_http_comp.c
+++ b/src/flt_http_comp.c
@@ -577,8 +577,6 @@ select_compression_response_header(struct comp_state *st, 
struct stream *s, stru
msg->flags |= HTTP_MSGF_COMPRESSING;
return 1;
 
-  deinit_comp_ctx:
-   st->comp_algo->end(&st->comp_ctx);
   fail:
st->comp_algo = NULL;
return 0;
-- 
2.24.1