Merge pull request #5 from nokk/master

replaced bignum with long
cainus · May 20, 2014 · f1bd25b · Raynos · Sep 5, 2014 · f1bd25b
2 parents 2d5ea25 + ad3b40b
commit f1bd25b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/BufferMaker.js b/lib/BufferMaker.js
@@ -1,4 +1,5 @@
-var bignum = require('bignum');
+var Long = require('long');
+
 
 var BufferMaker = function(){
   this.plan = [];
@@ -57,10 +58,9 @@ BufferMaker.prototype.make = function(){
     item = this.plan[i];
     switch(item.type){
       case "Int64BE":
-        var signed64BitBuffer = bignum(item.value).toBuffer({endian : "big", size : 8});
-        for(j = 0; j < signed64BitBuffer.length; j++){
-          buffer[offset + j] = signed64BitBuffer[j];
-        }
+        var longVal = Long.fromString(item.value  + "");
+        buffer.writeUInt32BE(longVal.getHighBits(), offset); //write the high order bits (shifted over)
+        buffer.writeUInt32BE(longVal.getLowBits(), offset + 4); //write the low order bits
         offset += 8;
         break;
       case "string": 

diff --git a/package.json b/package.json
@@ -21,7 +21,7 @@
         } 
       ],
       "dependencies": {
-          "bignum" : "0.6.2"
+          "long" : "latest"
       },
       "devDependencies" : {
           "jscoverage" : "0.3.6",