X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Ffreenet%2Fwot%2FDefaultIdentity.java;fp=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Ffreenet%2Fwot%2FDefaultIdentity.java;h=2ef33c3d3b5da14c9d8145fceed7210b89761882;hp=f200022b78b4f11895cb7e36e509230a1a90fa24;hb=17ed3b897e37c8b16c559b79bfe97d32a7605bfb;hpb=eb09646776e30665aa33e4a638ec3a4b3fe0632f

diff --git a/src/main/java/net/pterodactylus/sone/freenet/wot/DefaultIdentity.java b/src/main/java/net/pterodactylus/sone/freenet/wot/DefaultIdentity.java
index f200022..2ef33c3 100644
--- a/src/main/java/net/pterodactylus/sone/freenet/wot/DefaultIdentity.java
+++ b/src/main/java/net/pterodactylus/sone/freenet/wot/DefaultIdentity.java
@@ -17,14 +17,12 @@
 
 package net.pterodactylus.sone.freenet.wot;
 
+import java.util.Collection;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Map;
 import java.util.Set;
-import java.util.logging.Logger;
-
-import net.pterodactylus.util.logging.Logging;
 
 /**
  * A Web of Trust identity.
@@ -33,12 +31,6 @@ import net.pterodactylus.util.logging.Logging;
  */
 public class DefaultIdentity implements Identity {
 
-	/** The logger. */
-	private static final Logger logger = Logging.getLogger(DefaultIdentity.class);
-
-	/** The web of trust connector. */
-	private final WebOfTrustConnector webOfTrustConnector;
-
 	/** The ID of the identity. */
 	private final String id;
 
@@ -55,14 +47,11 @@ public class DefaultIdentity implements Identity {
 	private final Map<String, String> properties = Collections.synchronizedMap(new HashMap<String, String>());
 
 	/** Cached trust. */
-	/* synchronize on itself. */
-	private final Map<OwnIdentity, Trust> trustCache = new HashMap<OwnIdentity, Trust>();
+	private final Map<OwnIdentity, Trust> trustCache = Collections.synchronizedMap(new HashMap<OwnIdentity, Trust>());
 
 	/**
 	 * Creates a new identity.
 	 *
-	 * @param webOfTrustConnector
-	 *            The web of trust connector
 	 * @param id
 	 *            The ID of the identity
 	 * @param nickname
@@ -70,8 +59,7 @@ public class DefaultIdentity implements Identity {
 	 * @param requestUri
 	 *            The request URI of the identity
 	 */
-	public DefaultIdentity(WebOfTrustConnector webOfTrustConnector, String id, String nickname, String requestUri) {
-		this.webOfTrustConnector = webOfTrustConnector;
+	public DefaultIdentity(String id, String nickname, String requestUri) {
 		this.id = id;
 		this.nickname = nickname;
 		this.requestUri = requestUri;
@@ -114,47 +102,35 @@ public class DefaultIdentity implements Identity {
 	}
 
 	/**
-	 * Sets the contexts of this identity.
-	 * <p>
-	 * This method is only called by the {@link IdentityManager}.
-	 *
-	 * @param contexts
-	 *            The contexts to set
+	 * {@inheritDoc}
 	 */
-	void setContextsPrivate(Set<String> contexts) {
-		this.contexts.clear();
-		this.contexts.addAll(contexts);
+	@Override
+	public boolean hasContext(String context) {
+		return contexts.contains(context);
 	}
 
 	/**
 	 * {@inheritDoc}
 	 */
 	@Override
-	public boolean hasContext(String context) {
-		return contexts.contains(context);
+	public void setContexts(Collection<String> contexts) {
+		this.contexts.clear();
+		this.contexts.addAll(contexts);
 	}
 
 	/**
-	 * Adds the given context to this identity.
-	 * <p>
-	 * This method is only called by the {@link IdentityManager}.
-	 *
-	 * @param context
-	 *            The context to add
+	 * {@inheritDoc}
 	 */
-	void addContextPrivate(String context) {
+	@Override
+	public void addContext(String context) {
 		contexts.add(context);
 	}
 
 	/**
-	 * Removes the given context from this identity.
-	 * <p>
-	 * This method is only called by the {@link IdentityManager}.
-	 *
-	 * @param context
-	 *            The context to remove
+	 * {@inheritDoc}
 	 */
-	public void removeContextPrivate(String context) {
+	@Override
+	public void removeContext(String context) {
 		contexts.remove(context);
 	}
 
@@ -163,64 +139,40 @@ public class DefaultIdentity implements Identity {
 	 */
 	@Override
 	public Map<String, String> getProperties() {
-		synchronized (properties) {
-			return Collections.unmodifiableMap(properties);
-		}
+		return Collections.unmodifiableMap(properties);
 	}
 
 	/**
-	 * Sets all properties of this identity.
-	 * <p>
-	 * This method is only called by the {@link IdentityManager}.
-	 *
-	 * @param properties
-	 *            The new properties of this identity
+	 * {@inheritDoc}
 	 */
-	void setPropertiesPrivate(Map<String, String> properties) {
-		synchronized (this.properties) {
-			this.properties.clear();
-			this.properties.putAll(properties);
-		}
+	@Override
+	public void setProperties(Map<String, String> properties) {
+		this.properties.clear();
+		this.properties.putAll(properties);
 	}
 
 	/**
-	 * Sets the property with the given name to the given value.
-	 * <p>
-	 * This method is only called by the {@link IdentityManager}.
-	 *
-	 * @param name
-	 *            The name of the property
-	 * @param value
-	 *            The value of the property
+	 * {@inheritDoc}
 	 */
-	void setPropertyPrivate(String name, String value) {
-		synchronized (properties) {
-			properties.put(name, value);
-		}
+	@Override
+	public String getProperty(String name) {
+		return properties.get(name);
 	}
 
 	/**
 	 * {@inheritDoc}
 	 */
 	@Override
-	public String getProperty(String name) {
-		synchronized (properties) {
-			return properties.get(name);
-		}
+	public void setProperty(String name, String value) {
+		properties.put(name, value);
 	}
 
 	/**
-	 * Removes the property with the given name.
-	 * <p>
-	 * This method is only called by the {@link IdentityManager}.
-	 *
-	 * @param name
-	 *            The name of the property to remove
+	 * {@inheritDoc}
 	 */
-	void removePropertyPrivate(String name) {
-		synchronized (properties) {
-			properties.remove(name);
-		}
+	@Override
+	public void removeProperty(String name) {
+		properties.remove(name);
 	}
 
 	/**
@@ -228,23 +180,23 @@ public class DefaultIdentity implements Identity {
 	 */
 	@Override
 	public Trust getTrust(OwnIdentity ownIdentity) {
-		synchronized (trustCache) {
-			return trustCache.get(ownIdentity);
-		}
+		return trustCache.get(ownIdentity);
 	}
 
 	/**
-	 * Sets the trust received for this identity by the given own identity.
-	 *
-	 * @param ownIdentity
-	 *            The own identity that gives the trust
-	 * @param trust
-	 *            The trust received for this identity
+	 * {@inheritDoc}
 	 */
+	@Override
 	public void setTrust(OwnIdentity ownIdentity, Trust trust) {
-		synchronized (trustCache) {
-			trustCache.put(ownIdentity, trust);
-		}
+		trustCache.put(ownIdentity, trust);
+	}
+
+	/**
+	 * {@inheritDoc}
+	 */
+	@Override
+	public void removeTrust(OwnIdentity ownIdentity) {
+		trustCache.remove(ownIdentity);
 	}
 
 	//