From 455897cc245b67d929f409a93cfa7106e835cc1f Mon Sep 17 00:00:00 2001
From: boli <boli@localhost>
Date: Thu, 01 Mar 2007 03:27:06 +0000
Subject: [PATCH] This removes old debug logging framework method calls that are going to be automatically instrumented by AspectJ. Non instrumented debug method calls are updated to use the new debug framework methods. However, the new debug logging framework is not yet active as the Aspects are not weaved in. After this revision, debug logging will be disabled in the server until the new AOP framework is complete. 

---
 opends/src/server/org/opends/server/core/DefaultAccessControlProvider.java |   26 +-------------------------
 1 files changed, 1 insertions(+), 25 deletions(-)

diff --git a/opends/src/server/org/opends/server/core/DefaultAccessControlProvider.java b/opends/src/server/org/opends/server/core/DefaultAccessControlProvider.java
index 8d73c6a..d12d2c8 100644
--- a/opends/src/server/org/opends/server/core/DefaultAccessControlProvider.java
+++ b/opends/src/server/org/opends/server/core/DefaultAccessControlProvider.java
@@ -22,13 +22,10 @@
  * CDDL HEADER END
  *
  *
- *      Portions Copyright 2006 Sun Microsystems, Inc.
+ *      Portions Copyright 2006-2007 Sun Microsystems, Inc.
  */
 package org.opends.server.core;
 
-import static org.opends.server.loggers.Debug.debugConstructor;
-import static org.opends.server.loggers.Debug.debugEnter;
-
 import org.opends.server.api.AccessControlHandler;
 import org.opends.server.api.AccessControlProvider;
 import org.opends.server.config.ConfigEntry;
@@ -46,9 +43,6 @@
  * decision function which grants access to everything and anyone.
  */
 class DefaultAccessControlProvider implements AccessControlProvider {
-  // Fully qualified class name for debugging purposes.
-  private static final String CLASS_NAME =
-    "org.opends.server.core.DefaultAccessControlProvider";
 
   /**
    * The single handler instance.
@@ -61,7 +55,6 @@
   public DefaultAccessControlProvider() {
     super();
 
-    assert debugConstructor(CLASS_NAME);
 
     // No implementation required.
   }
@@ -71,7 +64,6 @@
    */
   public void initializeAccessControlHandler(ConfigEntry configEntry)
       throws ConfigException, InitializationException {
-    assert debugEnter(CLASS_NAME, "initializeAccessControlHandler");
 
     // Avoid potential race conditions constructing the handler instance
     // and create it here.
@@ -82,7 +74,6 @@
    * {@inheritDoc}
    */
   public void finalizeAccessControlHandler() {
-    assert debugEnter(CLASS_NAME, "finalizeAccessControlHandler");
 
     // No implementation required.
   }
@@ -91,7 +82,6 @@
    * {@inheritDoc}
    */
   public AccessControlHandler getInstance() {
-    assert debugEnter(CLASS_NAME, "getInstance");
 
     if (instance == null) {
       instance = new Handler();
@@ -104,16 +94,12 @@
    * control handler.
    */
   private static class Handler extends AccessControlHandler {
-    // Fully qualified class name for debugging purposes.
-    private static final String CLASS_NAME =
-      "org.opends.server.core.DefaultAccessControlProvider.Handler";
 
     /**
      * {@inheritDoc}
      */
     @Override
     public boolean isAllowed(AddOperation addOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -123,7 +109,6 @@
      */
     @Override
     public boolean isAllowed(BindOperation bindOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -133,7 +118,6 @@
      */
     @Override
     public boolean isAllowed(CompareOperation compareOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -143,7 +127,6 @@
      */
     @Override
     public boolean isAllowed(DeleteOperation deleteOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -153,7 +136,6 @@
      */
     @Override
     public boolean isAllowed(ExtendedOperation extendedOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -163,7 +145,6 @@
      */
     @Override
     public boolean isAllowed(ModifyOperation modifyOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -173,7 +154,6 @@
      */
     @Override
     public boolean isAllowed(ModifyDNOperation modifyDNOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -183,7 +163,6 @@
      */
     @Override
     public boolean isAllowed(SearchOperation searchOperation) {
-      assert debugEnter(CLASS_NAME, "isAllowed");
 
       return true;
     }
@@ -194,7 +173,6 @@
     @Override
     public boolean maySend(SearchOperation searchOperation,
         SearchResultEntry searchEntry) {
-      assert debugEnter(CLASS_NAME, "maySend");
 
       return true;
     }
@@ -205,7 +183,6 @@
     @Override
     public SearchResultEntry filterEntry(
         SearchOperation searchOperation, SearchResultEntry searchEntry) {
-      assert debugEnter(CLASS_NAME, "filterEntry");
 
       // No implementation required.
       return searchEntry;
@@ -217,7 +194,6 @@
     @Override
     public boolean maySend(SearchOperation searchOperation,
         SearchResultReference searchReference) {
-      assert debugEnter(CLASS_NAME, "maySend");
 
       return true;
     }

--
Gitblit v1.10.0