Sync readmemory not working about memoryjs

[p410n3]

I added a fix for findpattern too. Same problem

EDIT: Its in the pull req I made btw

[p410n3]

Here some more in depth explanation:

The module exports either the async or the synchronous version of the readmemory function here in the index.js of memoryJS:

readMemory(handle, address, dataType, callback) {
  if (arguments.length === 2) {
    return memoryjs.readMemory(handle, address, dataType.toLowerCase());
  }
  memoryjs.readMemory(handle, address, dataType.toLowerCase(), callback);
},

The error is here:
if (arguments.length === 2)
The previous version of memoryJS used only two arguments for the sync function and 3 for the async. So the 2 needs to be a 3, because it assumes I am trying to call the async function, which I am not.

And as you see, without my fix the examples inside the modules throws errors too.

at Object.<anonymous> (X:\Dokumente\p410n3.JS\node_modules\memoryjs\example1.js:86:61)
Given that, I know its an error in memoryJS.

You can confirm this by installing memoryJS and then doing:

cd ./node_modules/memoryjs
node example1.js

[LiamKarlMitchell]

Confirming, needs to be === 3

[p410n3]

https://github.com/p410n3/memoryjs/commit/da1c08f46c7bbb536580fc3d05f7c878674db449

[p410n3]

Fixed in https://github.com/Rob--/memoryjs/pull/24