You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by bt...@apache.org on 2020/07/17 02:24:28 UTC
[james-project] 14/31: [REFACTORING] Avoid variable reallocation in
PartialFetchBodyElement
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 <bt...@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-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org