More rigorous checking of optional arguments

[stt]

I was looking to do array filtering in a webworker and noticed that context was not set correctly as it though it was a list of imports.

By the way, although retrospectively obvious, it might be worth noting somewhere clearly that native functions can't be serialized by JSONfn. 😄
(I used the polyfill from https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/filter)

[vkiryukhin]

Hi stt,

Good catch!
I have a couple of comments:

try {
    return eval('(' + value + ')');
 } catch(e) {
    console.log(e);
    return value;
  }

1. would be nice to find more elegant way to implement try{...} part logic. Maybe you have any idea.
2. console.log() not needed in production of course.

I haven't work on this plugin for a while. If would useful if you provide most simple and most complex examples of object which you use for testing.

Thank you for working on plugin's improvement.

--Vadim

[stt]

This was probably meant as comment to #6 but only option for try-catch I see now is to do the "[native code]" string checking before calling or evaling it in JSONfn, either way it would duplicate couple of checks.
I'm not using the patch for real yet, but was looking to do something like this

var _filtered = ["asd","zxc"];
var fn = function filterBySomething(s) { return s=="asd"; };
vkthread.exec(_filtered.filter, [fn],
      function(data){
        _filtered = data;
      },
      _filtered);