[james-project] 24/30: [Refactoring] Set.add is idempotent in case of entry already present

[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 ca3555ec0e0b523003d11a42424dd686927b84b8
Author: Matthieu Baechler <matth...@apache.org>
AuthorDate: Wed Nov 13 16:22:32 2019 +0100

    [Refactoring] Set.add is idempotent in case of entry already present
---
 .../java/org/apache/james/mailrepository/file/FileMailRepository.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/server/data/data-file/src/main/java/org/apache/james/mailrepository/file/FileMailRepository.java
 
b/server/data/data-file/src/main/java/org/apache/james/mailrepository/file/FileMailRepository.java
index 98c6835..2472531 100644
--- 
a/server/data/data-file/src/main/java/org/apache/james/mailrepository/file/FileMailRepository.java
+++ 
b/server/data/data-file/src/main/java/org/apache/james/mailrepository/file/FileMailRepository.java
@@ -232,7 +232,7 @@ public class FileMailRepository implements MailRepository, 
Configurable, Initial
 
     private void internalStore(Mail mc) throws MessagingException, IOException 
{
         String key = mc.getName();
-        if (keys != null && !keys.contains(key)) {
+        if (keys != null) {
             keys.add(key);
         }
         boolean saveStream = true;


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