chrisa / node-dtrace-provider

Native DTrace probes for node.js apps
Other
320 stars 68 forks source link

foo.js contains PHP style assoc array instead of JSON #36

Closed ghost closed 11 years ago

ghost commented 11 years ago

inside foo.js (strangely I don't see this file in your repo, but it gets included when npm installs dtrace-provider@0.2.8) Line 6 returns a PHP style assoc array, which looks like it should be JSON?

was p1.fire(function() { return { "foo" => "bar" }; });

I modified mine to p1.fire(function() { return { "foo": "bar" }; });

How I found it: Netbeans was complaining about it, so I went to check it out.

chrisa commented 11 years ago

Argh! foo.js is just a scratch file lying around in my working copy, but must have been packaged up when I npm published version 0.2.8.

I'll make sure that the next version doesn't include anything like that :)

Thanks for letting me know.

ghost commented 11 years ago

groovy :)