andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Exception should be raised even if count is 0

Browse Source
This commit is contained in:
Trustin Lee 2010-04-12 11:57:46 +00:00
parent 318a4d901f
commit 4e69cfe281
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/org/jboss/netty/channel/DefaultFileRegion.java
View File

@ -31,14 +31,14 @@ public class DefaultFileRegion implements FileRegion {
public long transferTo(WritableByteChannel target, long position) throws IOException { public long transferTo(WritableByteChannel target, long position) throws IOException {
long count = this.count - position; long count = this.count - position;
if (count == 0) {
return 0L;
}
if (count < 0 || position < 0) { if (count < 0 || position < 0) {
throw new IllegalArgumentException( throw new IllegalArgumentException(
"position out of range: " + position + "position out of range: " + position +
" (expected: 0 - " + (this.count - 1) + ")"); " (expected: 0 - " + (this.count - 1) + ")");
} }
if (count == 0) {
return 0L;
}
return file.transferTo(this.position + position, count, target); return file.transferTo(this.position + position, count, target);
} }