From 8988b3f4c0ab891f31679d97d48dd376b4bd2fee Mon Sep 17 00:00:00 2001 From: Idel Pivnitskiy Date: Mon, 15 Sep 2014 18:30:00 +0400 Subject: [PATCH] Fixed typo in LzmaFrameEncoder constructor Motivation: A discovered typo in LzmaFrameEncoder constructor when we check `lc + lp` for better compatibility. Modifications: Changed `lc + pb` to `lc + lp`. Result: Correct check of `lc + lp` value. --- .../io/netty/handler/codec/compression/LzmaFrameEncoder.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/codec/src/main/java/io/netty/handler/codec/compression/LzmaFrameEncoder.java b/codec/src/main/java/io/netty/handler/codec/compression/LzmaFrameEncoder.java index f502983517..62cafc4e10 100644 --- a/codec/src/main/java/io/netty/handler/codec/compression/LzmaFrameEncoder.java +++ b/codec/src/main/java/io/netty/handler/codec/compression/LzmaFrameEncoder.java @@ -144,7 +144,7 @@ public class LzmaFrameEncoder extends MessageToByteEncoder { if (pb < 0 || pb > 4) { throw new IllegalArgumentException("pb: " + pb + " (expected: 0-4)"); } - if (lc + pb > 4) { + if (lc + lp > 4) { if (!warningLogged) { logger.warn("The latest versions of LZMA libraries (for example, XZ Utils) " + "has an additional requirement: lc + lp <= 4. Data which don't follow " +