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 2019/11/15 02:41:33 UTC
[james-project] 24/30: [Refactoring] Set.add is idempotent in case
of entry already present
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 <ma...@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-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org