From 8170a25a086fb89f90a2b2b8613e7b7d82a47208 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Thu, 08 Aug 2013 13:37:11 +0000
Subject: [PATCH] Big refactoring of ReplicationDB to make it readable. Changed count() return type from int to long. Removed a useless parameter from ReplicationIterator ctor. Various code cleanups in other classes.

---
 opends/src/server/org/opends/server/replication/server/DbHandler.java |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/opends/src/server/org/opends/server/replication/server/DbHandler.java b/opends/src/server/org/opends/server/replication/server/DbHandler.java
index 8231012..219204c 100644
--- a/opends/src/server/org/opends/server/replication/server/DbHandler.java
+++ b/opends/src/server/org/opends/server/replication/server/DbHandler.java
@@ -282,7 +282,7 @@
     {
       flush();
     }
-    return new ReplicationIterator(serverId, db, changeNumber, this);
+    return new ReplicationIterator(db, changeNumber, this);
   }
 
   /**
@@ -657,7 +657,7 @@
    * @param to   The upper (newer) change number.
    * @return The computed number of changes.
    */
-  public int getCount(ChangeNumber from, ChangeNumber to)
+  public long getCount(ChangeNumber from, ChangeNumber to)
   {
     // Now that we always keep the last ChangeNumber in the DB to avoid
     // expiring cookies too quickly, we need to check if the "to"

--
Gitblit v1.10.0