MonetDB: Dec2016 - BAT ids are positive, no need for abs.

[Sjoerd Mullender]

Changeset: f08081aa3791 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=f08081aa3791
Modified Files:
        gdk/gdk_batop.c
        gdk/gdk_logger.c
Branch: Dec2016
Log Message:

BAT ids are positive, no need for abs.


diffs (25 lines):

diff --git a/gdk/gdk_batop.c b/gdk/gdk_batop.c
--- a/gdk/gdk_batop.c
+++ b/gdk/gdk_batop.c
@@ -71,8 +71,7 @@ insert_string_bat(BAT *b, BAT *n, int fo
 
        assert(b->ttype == TYPE_str);
        /* only transient bats can use some other bat's string heap */
-       assert(b->batRole == TRANSIENT ||
-              b->tvheap->parentid == abs(b->batCacheid));
+       assert(b->batRole == TRANSIENT || b->tvheap->parentid == b->batCacheid);
        if (n->batCount == 0)
                return GDK_SUCCEED;
        ni = bat_iterator(n);
diff --git a/gdk/gdk_logger.c b/gdk/gdk_logger.c
--- a/gdk/gdk_logger.c
+++ b/gdk/gdk_logger.c
@@ -1249,7 +1249,7 @@ bm_subcommit(logger *lg, BAT *list_bid, 
                                        name,
                                        (list_bid == catalog_bid) ? 
BUNtvar(iter, p) : "snapshot");
                        assert(BBPindex(name));
-                       n[i++] = abs(BBPindex(name));
+                       n[i++] = BBPindex(name);
                }
        }
        /* now commit catalog, so it's also up to date on disk */
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list