Re: [PATCH] Make zlib compression level configurable, and change default.
- From
Linus Torvalds <torvalds@osdl.org>
- Date
- Jul 3, 2006, 19:50 UTC
- Message-ID
- <Pine.LNX.4.64.0607031248320.12404@g5.osdl.org>
- In-Reply-To
- <Pine.LNX.4.64.0607031226370.12404@g5.osdl.org>
On Mon, 3 Jul 2006, Linus Torvalds wrote:
> > Oh, and for all the same reasons, we should use > > int zlib_compression_level = Z_BEST_COMPRESSION;
That should be Z_DEFAULT_COMPRESSION, of course.
Anyway, I think the patches are ok as-is, and my suggestion to avoid the "-1" and use Z_DEFAULT_COMPRESSION is really just an additional comment, not anything fundamental.
So Junio, feel free to add an
Acked-by: Linus Torvalds <torvalds@osdl.org>
regardless of whether also doing that.
Linus