From 48a6c29fc9452226c348f4b476fd3b9a27e7d325 Mon Sep 17 00:00:00 2001 From: "Erik C. Thauvin" Date: Sat, 20 Apr 2019 05:22:06 -0700 Subject: [PATCH] Reworked deleting old message using removeIf instead of in a loop. --- .../thauvin/erik/mobibot/tell/TellMessagesMgr.java | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/src/main/java/net/thauvin/erik/mobibot/tell/TellMessagesMgr.java b/src/main/java/net/thauvin/erik/mobibot/tell/TellMessagesMgr.java index 4579c4c..fdddd99 100644 --- a/src/main/java/net/thauvin/erik/mobibot/tell/TellMessagesMgr.java +++ b/src/main/java/net/thauvin/erik/mobibot/tell/TellMessagesMgr.java @@ -75,17 +75,8 @@ final class TellMessagesMgr { */ static boolean clean(final List tellMessages, final int tellMaxDays) { final LocalDateTime today = LocalDateTime.now(Clock.systemUTC()); - boolean cleaned = false; - for (final TellMessage message : tellMessages) { - final LocalDateTime maxDate = message.getQueued().plusDays(tellMaxDays); - if (maxDate.isBefore(today)) { - tellMessages.remove(message); - cleaned = true; - } - } - - return cleaned; + return tellMessages.removeIf(o -> o.getQueued().plusDays(tellMaxDays).isBefore(today)); } /**