commit d44c714b4b4d3d89254dedfbec97115982423a8b
author Simone Tripodi <simonetripodi@apache.org> Fri Mar 15 08:56:13 2013 +0000
committer Simone Tripodi <simonetripodi@apache.org> Fri Mar 15 08:56:13 2013 +0000
tree 7d3efaaa7482097c43918dd4c627b8141697bf53
parent 084c24e56a31584cc8ffa5c47670451a2164b104

PMD: These nested if statements could be combined

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/fileupload/trunk@1456809 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/org/apache/commons/fileupload/FileUploadBase.java

diff --git a/src/main/java/org/apache/commons/fileupload/FileUploadBase.java b/src/main/java/org/apache/commons/fileupload/FileUploadBase.java
index 878012c..6114bd2 100644
--- a/src/main/java/org/apache/commons/fileupload/FileUploadBase.java
+++ b/src/main/java/org/apache/commons/fileupload/FileUploadBase.java
@@ -959,13 +959,11 @@
                                      : ctx.getContentLength();
 
             if (sizeMax >= 0) {
-                if (requestSize != -1) {
-                    if (requestSize > sizeMax) {
-                        throw new SizeLimitExceededException(
-                            format("the request was rejected because its size (%s) exceeds the configured maximum (%s)",
-                                   requestSize, sizeMax),
-                                   requestSize, sizeMax);
-                    }
+                if (requestSize != -1 && requestSize > sizeMax) {
+                    throw new SizeLimitExceededException(
+                        format("the request was rejected because its size (%s) exceeds the configured maximum (%s)",
+                               requestSize, sizeMax),
+                               requestSize, sizeMax);
                 }
                 input = new LimitedInputStream(input, sizeMax) {
                     @Override