[james-project] 14/31: [REFACTORING] Avoid variable reallocation in PartialFetchBodyElement

[btellier]

This is an automated email from the ASF dual-hosted git repository.

btellier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit a2ba8b1100485f63b5977aa78bb22dfdfd7e4940
Author: Benoit Tellier <btell...@linagora.com>
AuthorDate: Thu Jul 16 11:34:54 2020 +0700

    [REFACTORING] Avoid variable reallocation in PartialFetchBodyElement
---
 .../james/imap/processor/fetch/PartialFetchBodyElement.java       | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git 
a/protocols/imap/src/main/java/org/apache/james/imap/processor/fetch/PartialFetchBodyElement.java
 
b/protocols/imap/src/main/java/org/apache/james/imap/processor/fetch/PartialFetchBodyElement.java
index 0915a8d..78b3f8d 100644
--- 
a/protocols/imap/src/main/java/org/apache/james/imap/processor/fetch/PartialFetchBodyElement.java
+++ 
b/protocols/imap/src/main/java/org/apache/james/imap/processor/fetch/PartialFetchBodyElement.java
@@ -55,15 +55,13 @@ final class PartialFetchBodyElement implements BodyElement {
     public long size() throws IOException {
         final long size = delegate.size();
         final long lastOctet = this.numberOfOctets + firstOctet;
-        final long result;
         if (firstOctet > size) {
-            result = 0;
+            return  0;
         } else if (size > lastOctet) {
-            result = numberOfOctets;
+            return numberOfOctets;
         } else {
-            result = size - firstOctet;
+            return size - firstOctet;
         }
-        return result;
     }
 
     @Override


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
For additional commands, e-mail: server-dev-h...@james.apache.org