From 190ec89c2cb9902f765e8430362b6fef408d44fd Mon Sep 17 00:00:00 2001 From: Tom Hacohen Date: Thu, 20 Apr 2017 14:01:28 +0100 Subject: [PATCH] JournalModel: don't convert to list when getting collections. Just use the iterator we get back already. --- .../main/java/com/etesync/syncadapter/model/JournalModel.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/app/src/main/java/com/etesync/syncadapter/model/JournalModel.java b/app/src/main/java/com/etesync/syncadapter/model/JournalModel.java index fa19b868..f4320bb1 100644 --- a/app/src/main/java/com/etesync/syncadapter/model/JournalModel.java +++ b/app/src/main/java/com/etesync/syncadapter/model/JournalModel.java @@ -54,8 +54,7 @@ public class JournalModel { public static List getCollections(EntityDataStore data, long service) { List ret = new LinkedList<>(); - List journals = data.select(JournalEntity.class).where(JournalEntity.SERVICE.eq(service).and(JournalEntity.DELETED.eq(false))).get().toList(); - for (JournalEntity journal : journals) { + for (JournalEntity journal : data.select(JournalEntity.class).where(JournalEntity.SERVICE.eq(service).and(JournalEntity.DELETED.eq(false))).get()) { // FIXME: For some reason this isn't always being called, manually do it here. journal.afterLoad(); ret.add(journal.getInfo());