You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2013/03/07 15:12:04 UTC
svn commit: r1453878 -
/commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java
Author: simonetripodi
Date: Thu Mar 7 14:12:04 2013
New Revision: 1453878
URL: http://svn.apache.org/r1453878
Log:
PMD: These nested if statements could be combined
Modified:
commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java
Modified: commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java
URL: http://svn.apache.org/viewvc/commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java?rev=1453878&r1=1453877&r2=1453878&view=diff
==============================================================================
--- commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java (original)
+++ commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java Thu Mar 7 14:12:04 2013
@@ -862,10 +862,8 @@ public class MultipartStream {
if (closed) {
throw new FileItemStream.ItemSkippedException();
}
- if (available() == 0) {
- if (makeAvailable() == 0) {
- return -1;
- }
+ if (available() == 0 && makeAvailable() == 0) {
+ return -1;
}
++total;
int b = buffer[head++];