itable: re-enable replace already existing value

This is a possible memory leak.

Author: btovar

dttools/src/itable.c

@@ -131,19 +131,9 @@ double itable_load(struct itable *h)
 static int insert_to_buckets_aux(struct entry **buckets, int bucket_count, struct entry *new_entry)
 {
 	unsigned index;
-	struct entry *e;
-
 	index = new_entry->key % bucket_count;
-	e = buckets[index];
-
-	while (e) {
-		/* check that this key does not already exist in the table */
-		if (new_entry->key == e->key) {
-			return 0;
-		}
-		e = e->next;
-	}
 
+	// Possible memory leak! Silently replacing value if it existed.
 	new_entry->next = buckets[index];
 	buckets[index] = new_entry;
 	return 1;

dttools/src/itable.h

@@ -103,8 +103,7 @@ double itable_load(struct itable *h);
 
 
 /** Insert a key and value.
-This call will fail if the table already contains the same key.
-You must call @ref itable_remove to remove it.
+This call will replace the value if it already contains the same key.
 Also note that you cannot insert a null value into the table.
 @param h A pointer to an integer table.
 @param key An integer key