[james-project] 11/21: [Refactoring] ListProcessor: Avoid variable re-allocation in getMailboxType

[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 41be21f65b98eb2bd851e1321ae448da9530e17b
Author: Benoit Tellier <btell...@linagora.com>
AuthorDate: Tue Nov 12 15:54:18 2019 +0700

    [Refactoring] ListProcessor: Avoid variable re-allocation in getMailboxType
---
 .../src/main/java/org/apache/james/imap/processor/ListProcessor.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/protocols/imap/src/main/java/org/apache/james/imap/processor/ListProcessor.java
 
b/protocols/imap/src/main/java/org/apache/james/imap/processor/ListProcessor.java
index 851aa94..67fbf70 100644
--- 
a/protocols/imap/src/main/java/org/apache/james/imap/processor/ListProcessor.java
+++ 
b/protocols/imap/src/main/java/org/apache/james/imap/processor/ListProcessor.java
@@ -187,11 +187,10 @@ public class ListProcessor extends 
AbstractMailboxProcessor<ListRequest> {
      * @return MailboxType value
      */
     private MailboxType getMailboxType(ImapSession session, MailboxTyper 
mailboxTyper, MailboxPath path) {
-        MailboxType result = MailboxType.OTHER;
         if (mailboxTyper != null) {
-            result = mailboxTyper.getMailboxType(session, path);
+            return mailboxTyper.getMailboxType(session, path);
         }
-        return result;
+        return MailboxType.OTHER;
     }
 
     @Override


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