From b944531b5e2ff68cbd26a73455b20e4e4a61ee9d Mon Sep 17 00:00:00 2001 From: Trustin Lee Date: Sat, 22 Oct 2011 21:51:38 -0700 Subject: [PATCH] Simpler constructor form should appear first --- .../jboss/netty/channel/DefaultFileRegion.java | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/main/java/org/jboss/netty/channel/DefaultFileRegion.java b/src/main/java/org/jboss/netty/channel/DefaultFileRegion.java index 5d0c93ecc3..d4f335211e 100644 --- a/src/main/java/org/jboss/netty/channel/DefaultFileRegion.java +++ b/src/main/java/org/jboss/netty/channel/DefaultFileRegion.java @@ -16,13 +16,6 @@ public class DefaultFileRegion implements FileRegion { private final long count; private final boolean releaseAfterTransfer; - public DefaultFileRegion(FileChannel file, long position, long count, boolean releaseAfterTransfer) { - this.file = file; - this.position = position; - this.count = count; - this.releaseAfterTransfer = releaseAfterTransfer; - } - /** * Calls {@link #DefaultFileRegion(FileChannel, long, long, boolean)} * with true as the last argument. @@ -30,7 +23,13 @@ public class DefaultFileRegion implements FileRegion { public DefaultFileRegion(FileChannel file, long position, long count) { this(file, position, count, true); } - + + public DefaultFileRegion(FileChannel file, long position, long count, boolean releaseAfterTransfer) { + this.file = file; + this.position = position; + this.count = count; + this.releaseAfterTransfer = releaseAfterTransfer; + } @Override public long getPosition() {