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 1ff0539d8354ecb5f5931f1265887e3c3d57910a Author: Benoit Tellier <[email protected]> AuthorDate: Fri Nov 15 11:05:27 2019 +0700 [Refactoring] MessageRangeProcessor: avoid final variables --- .../james/imap/processor/AbstractMessageRangeProcessor.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractMessageRangeProcessor.java b/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractMessageRangeProcessor.java index d26eca9..22fe645 100644 --- a/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractMessageRangeProcessor.java +++ b/protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractMessageRangeProcessor.java @@ -51,19 +51,19 @@ public abstract class AbstractMessageRangeProcessor<R extends AbstractMessageRan super(acceptableClass, next, mailboxManager, factory, metricFactory); } - protected abstract List<MessageRange> process(final MailboxPath targetMailbox, - final SelectedMailbox currentMailbox, - final MailboxSession mailboxSession, + protected abstract List<MessageRange> process(MailboxPath targetMailbox, + SelectedMailbox currentMailbox, + MailboxSession mailboxSession, MessageRange messageSet) throws MailboxException; protected abstract String getOperationName(); @Override protected void processRequest(R request, ImapSession session, Responder responder) { - final MailboxPath targetMailbox = PathConverter.forSession(session).buildFullPath(request.getMailboxName()); + MailboxPath targetMailbox = PathConverter.forSession(session).buildFullPath(request.getMailboxName()); try { - final MailboxSession mailboxSession = ImapSessionUtils.getMailboxSession(session); + MailboxSession mailboxSession = ImapSessionUtils.getMailboxSession(session); if (!getMailboxManager().mailboxExists(targetMailbox, mailboxSession)) { no(request, responder, HumanReadableText.FAILURE_NO_SUCH_MAILBOX, StatusResponse.ResponseCode.tryCreate()); @@ -84,7 +84,7 @@ public abstract class AbstractMessageRangeProcessor<R extends AbstractMessageRan } private StatusResponse.ResponseCode handleRanges(R request, ImapSession session, MailboxPath targetMailbox, MailboxSession mailboxSession) throws MailboxException { - final MessageManager mailbox = getMailboxManager().getMailbox(targetMailbox, mailboxSession); + MessageManager mailbox = getMailboxManager().getMailbox(targetMailbox, mailboxSession); List<IdRange> resultRanges = new ArrayList<>(); for (IdRange range : request.getIdSet()) { --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
