Index: /Users/cziegeler/Developer/workspaces/default/jackrabbit-sandbox/jackrabbit-jcr-mapping/jcr-mapping/src/test/java/org/apache/jackrabbit/ocm/security/SimpleAccessManager.java
===================================================================
--- /Users/cziegeler/Developer/workspaces/default/jackrabbit-sandbox/jackrabbit-jcr-mapping/jcr-mapping/src/test/java/org/apache/jackrabbit/ocm/security/SimpleAccessManager.java	(revision 590614)
+++ /Users/cziegeler/Developer/workspaces/default/jackrabbit-sandbox/jackrabbit-jcr-mapping/jcr-mapping/src/test/java/org/apache/jackrabbit/ocm/security/SimpleAccessManager.java	(working copy)
@@ -28,16 +28,14 @@
 import org.apache.jackrabbit.core.security.AccessManager;
 import org.apache.jackrabbit.core.security.AnonymousPrincipal;
 import org.apache.jackrabbit.core.security.SystemPrincipal;
-import org.apache.log4j.Logger;
 
+
 /**
  * <code>SimpleAccessManager</code> ...
  */
 public class SimpleAccessManager implements AccessManager
 {
 
-	private static Logger log = Logger.getLogger(SimpleAccessManager.class);
-
 	/**
 	 * Subject whose access rights this AccessManager should reflect
 	 */
@@ -149,8 +147,8 @@
 
 		// @todo check permission based on principals
 		return true;
-	
-		
+
+
 	}
 
 	/**
Index: /Users/cziegeler/Developer/workspaces/default/jackrabbit-sandbox/jackrabbit-jcr-mapping/jcr-mapping/src/test/java/org/apache/jackrabbit/ocm/security/SimpleLoginModule.java
===================================================================
--- /Users/cziegeler/Developer/workspaces/default/jackrabbit-sandbox/jackrabbit-jcr-mapping/jcr-mapping/src/test/java/org/apache/jackrabbit/ocm/security/SimpleLoginModule.java	(revision 590614)
+++ /Users/cziegeler/Developer/workspaces/default/jackrabbit-sandbox/jackrabbit-jcr-mapping/jcr-mapping/src/test/java/org/apache/jackrabbit/ocm/security/SimpleLoginModule.java	(working copy)
@@ -34,7 +34,6 @@
 import org.apache.jackrabbit.core.security.CredentialsCallback;
 import org.apache.jackrabbit.core.security.SecurityConstants;
 import org.apache.jackrabbit.core.security.UserPrincipal;
-import org.apache.log4j.Logger;
 
 /**
  * A <code>SimpleLoginModule</code> ...
@@ -42,8 +41,6 @@
 public class SimpleLoginModule implements LoginModule
 {
 
-	private static Logger log = Logger.getLogger(SimpleLoginModule.class);
-
 	/**
 	 * Name of the anonymous user id option in the LoginModule configuration
 	 */
@@ -134,8 +131,6 @@
 					Object attr = sc.getAttribute(SecurityConstants.IMPERSONATOR_ATTRIBUTE);
 					if (attr != null && attr instanceof Subject)
 					{
-						Subject impersonator = (Subject) attr;
-						// @todo check privileges to 'impersonate' the user represented by the supplied credentials
 					}
 					else
 					{