Set the default lgwin value in the encoder constructor.

895dcc8d · Geoff Simmons · 9f229baa · 895dcc8d · 895dcc8d · 895dcc8d
Commit 895dcc8d authored Feb 15, 2019 by Geoff Simmons
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 4 deletions

README.rst README.rst +1 -1

vfp_brotli.c src/vfp_brotli.c +1 -2

vfp_brotli.vcc src/vfp_brotli.vcc +1 -1

No files found.
--- a/README.rst
+++ b/README.rst
@@ -78,7 +78,7 @@ new xencoder = brotli.encoder(STRING name, BYTES bufffer, INT quality, BOOL larg
      BYTES bufffer=32768,
      INT quality=11,
      BOOL large_win=0,
-      INT lgwin=0
+      INT lgwin=22
   )

 XXX ...

--- a/src/vfp_brotli.c
+++ b/src/vfp_brotli.c
@@ -772,8 +772,7 @@ vmod_encoder__init(VRT_CTX, struct vmod_brotli_encoder **encp,
 		return;
 	}

-	if (lgwin != 0 &&
-	    (lgwin < BROTLI_MIN_WINDOW_BITS || lgwin > BROTLI_MAX_WINDOW_BITS)) {
+	if (lgwin < BROTLI_MIN_WINDOW_BITS || lgwin > BROTLI_MAX_WINDOW_BITS) {
 		VFAIL(ctx, "new %s: lgwin %ld out of range (%d to %d)",
 		      vcl_name, lgwin, BROTLI_MIN_WINDOW_BITS,
 		      BROTLI_MAX_WINDOW_BITS);

--- a/src/vfp_brotli.vcc
+++ b/src/vfp_brotli.vcc
@@ -63,7 +63,7 @@ algorithm for responses fetched from backends.
 XXX ...

 $Object encoder(PRIV_VCL, STRING name, BYTES bufffer=32768, INT quality=11,
-                BOOL large_win=0, INT lgwin=0)
+                BOOL large_win=0, INT lgwin=22)

 XXX ...