From 1e58ebd4607490d810c7396a4e87eb70d9528166 Mon Sep 17 00:00:00 2001
From: Adam Dickmeiss <adam@indexdata.dk>
Date: Sun, 2 Jan 2005 18:51:31 +0000
Subject: [PATCH] Swap functions - minor changes to compare tst function

---
 isamb/tstisamb.c |  149 +++++++++++++++++++++++++++++-------------------------
 1 file changed, 80 insertions(+), 69 deletions(-)

diff --git a/isamb/tstisamb.c b/isamb/tstisamb.c
index 67e3336..17f2a38 100644
--- a/isamb/tstisamb.c
+++ b/isamb/tstisamb.c
@@ -1,5 +1,5 @@
-/* $Id: tstisamb.c,v 1.13 2004-12-13 20:51:31 adam Exp $
-   Copyright (C) 1995,1996,1997,1998,1999,2000,2001,2002,2003,2004
+/* $Id: tstisamb.c,v 1.14 2005-01-02 18:51:31 adam Exp $
+   Copyright (C) 1995-2005
    Index Data Aps
 
 This file is part of the Zebra server.
@@ -44,7 +44,11 @@ int compare_item(const void *a, const void *b)
 
     memcpy(&ia, a, sizeof(int));
     memcpy(&ib, b, sizeof(int));
-    return ia - ib;
+    if (ia > ib)
+	return 1;
+    if (ia < ib)
+	return -1;
+   return 0;
 }
 
 void *code_start()
@@ -84,67 +88,11 @@ int code_read(void *vp, char **dst, int *insertMode)
     memcpy (*dst, &x, sizeof(int));
     (*dst)+=sizeof(int);
 
-    ri->no += ri->step;
+    ri->no = ri->no + ri->step;
     *insertMode = ri->insertMode;
     return 1;
 }
 
-void tst_forward(ISAMB isb, int n)
-{
-    ISAMC_I isamc_i;
-    ISAMC_P isamc_p;
-    struct read_info ri;
-    int i;
-    ISAMB_PP pp;
-
-    /* insert a number of entries */
-    ri.no = 0;
-    ri.step = 1;
-    ri.max = n;
-    ri.insertMode = 1;
-
-    isamc_i.clientData = &ri;
-    isamc_i.read_item = code_read;
-    
-    isamc_p = isamb_merge (isb, 0 /* new list */ , &isamc_i);
-
-    /* read the entries */
-    pp = isamb_pp_open (isb, isamc_p, 2);
-    
-    for (i = 0; i<ri.max; i +=2 )
-    {
-	int x = -1;
-	int xu = i;
-	isamb_pp_forward(pp, &x, &xu);
-	if (x != xu && xu != x+1)
-	{
-	    yaz_log(YLOG_WARN, "isamb_pp_forward (1). Got %d (expected %d)",
-		    x, xu);
-	    exit(4);
-	}
-	ri.no++;
-    }
-    isamb_pp_close(pp);
-    
-    pp = isamb_pp_open (isb, isamc_p, 2);
-    for (i = 0; i<ri.max; i += 100)
-    {
-	int x = -1;
-	int xu = i;
-	isamb_pp_forward(pp, &x, &xu);
-	if (x != xu && xu != x+1)
-	{
-	    yaz_log(YLOG_WARN, "isamb_pp_forward (2). Got %d (expected %d)",
-		    x, xu);
-	    exit(4);
-	}
-	ri.no++;
-    }
-    isamb_pp_close(pp);
-
-    isamb_unlink(isb, isamc_p);
-}
-
 void tst_insert(ISAMB isb, int n)
 {
     ISAMC_I isamc_i;
@@ -152,6 +100,7 @@ void tst_insert(ISAMB isb, int n)
     struct read_info ri;
     ISAMB_PP pp;
     char key_buf[10];
+    int nerrs = 0;
 
     /* insert a number of entries */
     ri.no = 0;
@@ -165,8 +114,8 @@ void tst_insert(ISAMB isb, int n)
     isamc_p = isamb_merge (isb, 0 /* new list */ , &isamc_i);
 
     /* read the entries */
-    pp = isamb_pp_open (isb, isamc_p, 2);
-    
+    pp = isamb_pp_open (isb, isamc_p, 1);
+
     ri.no = 0;
     while(isamb_pp_read (pp, key_buf))
     {
@@ -174,21 +123,27 @@ void tst_insert(ISAMB isb, int n)
 	memcpy (&x, key_buf, sizeof(int));
 	if (x != ri.no)
 	{
-	    yaz_log(YLOG_WARN, "isamb_pp_read. Got %d (expected %d)",
-		    x, ri.no);
-	    exit(3);
+	    yaz_log(YLOG_WARN, "isamb_pp_read. n=%d Got %d (expected %d)",
+		    n, x, ri.no);
+	    nerrs++;
 	}
+	else if (nerrs)
+	    yaz_log(YLOG_LOG, "isamb_pp_read. n=%d Got %d",
+		    n, x);
+
 	ri.no++;
     }
     if (ri.no != ri.max)
     {
 	yaz_log(YLOG_WARN, "ri.max != ri.max (%d != %d)", ri.no, ri.max);
-	exit(3);
+	nerrs++;
     }
-    isamb_pp_close(pp);
-
     isamb_dump(isb, isamc_p, log_pr);
+    isamb_pp_close(pp);
 
+    if (nerrs)
+        exit(3);
+    return;
     /* delete a number of entries (even ones) */
     ri.no = 0;
     ri.step = 2;
@@ -201,7 +156,7 @@ void tst_insert(ISAMB isb, int n)
     isamc_p = isamb_merge (isb, isamc_p , &isamc_i);
 
     /* delete a number of entries (odd ones) */
-    ri.no = 1;
+    ri.no = 0;
     ri.step = 2;
     ri.max = n;
     ri.insertMode = 0;
@@ -218,6 +173,62 @@ void tst_insert(ISAMB isb, int n)
     }
 }
 
+void tst_forward(ISAMB isb, int n)
+{
+    ISAMC_I isamc_i;
+    ISAMC_P isamc_p;
+    struct read_info ri;
+    int i;
+    ISAMB_PP pp;
+
+    /* insert a number of entries */
+    ri.no = 0;
+    ri.step = 1;
+    ri.max = n;
+    ri.insertMode = 1;
+
+    isamc_i.clientData = &ri;
+    isamc_i.read_item = code_read;
+    
+    isamc_p = isamb_merge (isb, 0 /* new list */ , &isamc_i);
+
+    /* read the entries */
+    pp = isamb_pp_open (isb, isamc_p, 1);
+    
+    for (i = 0; i<ri.max; i +=2 )
+    {
+	int x = -1;
+	int xu = i;
+	isamb_pp_forward(pp, &x, &xu);
+	if (x != xu && xu != x+1)
+	{
+	    yaz_log(YLOG_WARN, "isamb_pp_forward (1). Got %d (expected %d)",
+		    x, xu);
+	    exit(4);
+	}
+	ri.no++;
+    }
+    isamb_pp_close(pp);
+    
+    pp = isamb_pp_open (isb, isamc_p, 1);
+    for (i = 0; i<ri.max; i += 100)
+    {
+	int x = -1;
+	int xu = i;
+	isamb_pp_forward(pp, &x, &xu);
+	if (x != xu && xu != x+1)
+	{
+	    yaz_log(YLOG_WARN, "isamb_pp_forward (2). Got %d (expected %d)",
+		    x, xu);
+	    exit(4);
+	}
+	ri.no++;
+    }
+    isamb_pp_close(pp);
+
+    isamb_unlink(isb, isamc_p);
+}
+
 int main(int argc, char **argv)
 {
     BFiles bfs;
-- 
1.7.10.4