From a415256cc798baf286d618a2c9ddd427c26a518d Mon Sep 17 00:00:00 2001
From: matthew_swift <matthew_swift@localhost>
Date: Mon, 14 Dec 2009 13:21:39 +0000
Subject: [PATCH] Remove optional P parameter from result handlers as it is hardly ever needed in practice and just pollutes the APIs.

---
 sdk/src/com/sun/opends/sdk/ldap/SearchResultFutureImpl.java |   23 ++++++++++-------------
 1 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/sdk/src/com/sun/opends/sdk/ldap/SearchResultFutureImpl.java b/sdk/src/com/sun/opends/sdk/ldap/SearchResultFutureImpl.java
index 31da817..5266d71 100644
--- a/sdk/src/com/sun/opends/sdk/ldap/SearchResultFutureImpl.java
+++ b/sdk/src/com/sun/opends/sdk/ldap/SearchResultFutureImpl.java
@@ -46,32 +46,29 @@
 /**
  * Search result future implementation.
  */
-public final class SearchResultFutureImpl<P> extends
-    AbstractResultFutureImpl<Result, P> implements ResultFuture<Result>
+public final class SearchResultFutureImpl extends
+    AbstractResultFutureImpl<Result> implements ResultFuture<Result>
 {
 
-  private final SearchResultHandler<P> searchResultHandler;
-
-  private final P p;
+  private final SearchResultHandler searchResultHandler;
 
   private final SearchRequest request;
 
 
 
   SearchResultFutureImpl(int messageID, SearchRequest request,
-      ResultHandler<Result, P> resultHandler,
-      SearchResultHandler<P> searchResultHandler, P p,
+      ResultHandler<Result> resultHandler,
+      SearchResultHandler searchResultHandler,
       LDAPConnection connection, ExecutorService handlerExecutor)
   {
-    super(messageID, resultHandler, p, connection, handlerExecutor);
+    super(messageID, resultHandler, connection, handlerExecutor);
     this.request = request;
     this.searchResultHandler = searchResultHandler;
-    this.p = p;
   }
 
 
 
-  synchronized void handleSearchResultEntry(
+  void handleSearchResultEntry(
       final SearchResultEntry entry)
   {
     if (!isDone())
@@ -82,7 +79,7 @@
         {
           public void run()
           {
-            searchResultHandler.handleEntry(p, entry);
+            searchResultHandler.handleEntry(entry);
           }
         });
       }
@@ -91,7 +88,7 @@
 
 
 
-  synchronized void handleSearchResultReference(
+  void handleSearchResultReference(
       final SearchResultReference reference)
   {
     if (!isDone())
@@ -102,7 +99,7 @@
         {
           public void run()
           {
-            searchResultHandler.handleReference(p, reference);
+            searchResultHandler.handleReference(reference);
           }
         });
       }

--
Gitblit v1.10.0