Fix Map hash bug (#281)

- `map_hash_key` must generate the same key for JS_INT and JS_FLOAT64
   with the same value
- add test cases in tests/test_builtin.js

Author: chqrlie

quickjs.c

@@ -44377,7 +44377,7 @@ static uint32_t map_hash_key(JSContext *ctx, JSValue key)
         h = (uintptr_t)JS_VALUE_GET_PTR(key) * 3163;
         break;
     case JS_TAG_INT:
-        d = JS_VALUE_GET_INT(key) * 3163;
+        d = JS_VALUE_GET_INT(key);
         goto hash_float64;
     case JS_TAG_BIG_INT:
         a = JS_GetBigInt(key);
@@ -44391,7 +44391,7 @@ static uint32_t map_hash_key(JSContext *ctx, JSValue key)
     hash_float64:
         u.d = d;
         h = (u.u32[0] ^ u.u32[1]) * 3163;
-        break;
+        return h ^= JS_TAG_FLOAT64;
     default:
         h = 0;
         break;

tests/test_builtin.js

@@ -737,6 +737,16 @@ function test_map()
 {
     var a, i, n, tab, o, v;
     n = 1000;
+
+    a = new Map();
+    for (var i = 0; i < n; i++) {
+        a.set(i, i);
+    }
+    a.set(-2147483648, 1);
+    assert(a.get(-2147483648), 1);
+    assert(a.get(-2147483647 - 1), 1);
+    assert(a.get(-2147483647.5 - 0.5), 1);
+
     a = new Map();
     tab = [];
     for(i = 0; i < n; i++) {