X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fdata%2Fimpl%2FSoneImpl.java;h=24792763c471032b345278998095d8ab3ae97cc0;hb=90d193f69c4aa12c232f937a9d2b7d5afdda81dd;hp=6da31fc655159ccc1ba9914829dee76df51f2e0d;hpb=3752ebf20fcb0c71287f2b7fed358b14c7649d7f;p=Sone.git

diff --git a/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java b/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
index 6da31fc..2479276 100644
--- a/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
+++ b/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
@@ -25,6 +25,7 @@ import java.net.MalformedURLException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import java.util.concurrent.CopyOnWriteArraySet;
@@ -47,6 +48,7 @@ import net.pterodactylus.sone.freenet.wot.OwnIdentity;
 
 import freenet.keys.FreenetURI;
 
+import com.google.common.collect.FluentIterable;
 import com.google.common.hash.Hasher;
 import com.google.common.hash.Hashing;
 
@@ -93,7 +95,7 @@ public class SoneImpl implements LocalSone {
 	private volatile boolean known;
 
 	/** All posts. */
-	private final Set<Post> posts = new CopyOnWriteArraySet<Post>();
+	private final Collection<Post> posts = new HashSet<Post>();
 
 	/** All replies. */
 	private final Set<PostReply> replies = new CopyOnWriteArraySet<PostReply>();
@@ -119,13 +121,14 @@ public class SoneImpl implements LocalSone {
 	 * @param local
 	 * 		{@code true} if the Sone is a local Sone, {@code false} otherwise
 	 */
-	public SoneImpl(Database database, Identity identity, boolean local, long time, Client client) {
+	public SoneImpl(Database database, Identity identity, boolean local, long time, Client client, Collection<Post> posts) {
 		this.database = database;
 		this.id = identity.getId();
 		this.identity = identity;
 		this.local = local;
 		this.time = time;
 		this.client = client;
+		this.posts.addAll(posts);
 	}
 
 	//
@@ -354,12 +357,9 @@ public class SoneImpl implements LocalSone {
 	 * @return All posts of this Sone
 	 */
 	public List<Post> getPosts() {
-		List<Post> sortedPosts;
 		synchronized (this) {
-			sortedPosts = new ArrayList<Post>(posts);
+			return FluentIterable.from(posts).toSortedList(Post.TIME_COMPARATOR);
 		}
-		Collections.sort(sortedPosts, Post.TIME_COMPARATOR);
-		return sortedPosts;
 	}
 
 	/**
@@ -370,10 +370,6 @@ public class SoneImpl implements LocalSone {
 	 * @return This Sone (for method chaining)
 	 */
 	public Sone setPosts(Collection<Post> posts) {
-		synchronized (this) {
-			this.posts.clear();
-			this.posts.addAll(posts);
-		}
 		return this;
 	}