Browse Source

Don't clear the HashMap, instead just change the reference

master
Anthony Restaino 9 years ago
parent
commit
dbf0457d79
  1. 8
      app/src/main/java/acr/browser/lightning/database/BookmarkManager.java

8
app/src/main/java/acr/browser/lightning/database/BookmarkManager.java

@ -51,7 +51,7 @@ public class BookmarkManager {
private final String DEFAULT_BOOKMARK_TITLE; private final String DEFAULT_BOOKMARK_TITLE;
private final Map<String, HistoryItem> mBookmarksMap = new HashMap<>(); private Map<String, HistoryItem> mBookmarksMap;
// private final List<HistoryItem> mBookmarkList = new ArrayList<>(); // private final List<HistoryItem> mBookmarkList = new ArrayList<>();
private String mCurrentFolder = ""; private String mCurrentFolder = "";
private final ExecutorService mExecutor; private final ExecutorService mExecutor;
@ -131,8 +131,7 @@ public class BookmarkManager {
} finally { } finally {
Utils.close(bookmarksReader); Utils.close(bookmarksReader);
} }
mBookmarksMap.clear(); mBookmarksMap = bookmarks;
mBookmarksMap.putAll(bookmarks);
mReady = true; mReady = true;
} }
} }
@ -283,8 +282,7 @@ public class BookmarkManager {
bookmarks.put(url, item); bookmarks.put(url, item);
} }
} }
mBookmarksMap.clear(); mBookmarksMap = bookmarks;
mBookmarksMap.putAll(bookmarks);
mExecutor.execute(new BookmarksWriter(new LinkedList<>(mBookmarksMap.values()))); mExecutor.execute(new BookmarksWriter(new LinkedList<>(mBookmarksMap.values())));
} }

Loading…
Cancel
Save