Small bugs when parsing code

[stuta]

Hello. Great project!

http://tylerneylon.com/a/learn-lua/

u = {['@!#'] = 'qbert', [{}] = 1729, [6.28] = 'tau'}
print(u[6.28]) -- prints "tau"

I had to comment 2 parts to get this file to run:
u = {['@!#'] = 'qbert', [6.28] = 'tau'} -- , [{}] = 1729
-- print(u[6.28]) -- prints "tau"

Also os.clock() returns nil. I was able to get timing to wor with this js:
var timer = function () {
var d=new Date();
return d.getTime();
}

[paulcuth]

Hi,

Many thanks for the feedback.

That snippet of code from your site actually highlighted two issues: one in tforloop and another in GC reference counting. Both were related to using tables as keys and both have now been fixed in 109c0ac.

A lot of the os library is currently unimplemented due to the fact that the Web doesn't translate easily to the typical model of an os. The clock() function does appear to be one that could be implemented, but I'm unfamiliar with its behaviour. If clock() is called in a callback or event listener, would you expect it to return the time since the script was first executed or from when the callback was triggered?

Thanks again for your feedback, the more the better. :)

[agladysh]

os.clock() actually behaves differently, depending on implementation and OS. We ban its use in our production Lua code at LogicEditor (not related to this project). I, personally, don't think that it is worth creating yet another incompatible implementation of os.clock. Better to clearly signal users to roll their own as needed (preferably without modifying os table, of course).

My 2c.

[stuta]

Great, code works now.

How can I redirect os.clock to js function?

This does not work:

var timer = function () {
var d=new Date();
return d.getTime()/1000;
}
vm = new shine.VM({
os.clock: timer
});

There are lots of lua/js code in mherkender/lua.js#35 if you are missing something.

[paulcuth]

I agree with Alexander and this looks like a different implementation again. However, if that's what you want to do, try this:

var timer = function () { return Date.now(); },
    vm, os;

vm = new shine.VM();
os = vm.getGlobal('os');    
os.setMember('clock', timer);

Cheers, for the heads-up on the recent updates to lua.js, although Moonshine has all that functionality already. :)

Let me know if Moonshine can be of any more use to your site. Have you seen the new editor (http://moonshinejs.org/editor/)? I thinking of making it embeddable in other sites, might that be of use to you?

[paulcuth]

Actually, it would be easier to override the lib function before instantiating the VM:

shine.lib.os.clock = function () { return Date.now(); };
var vm = new shine.VM();

[stuta]

I tried to run http://luajit.org/download/scimark.lua. It seems that string.format does not work.

SciMark result in Safari 0.106 is not good. Fierfox is much worse: 0.046. Results are far away from http://kripken.github.io/lua.vm.js/lua.vm.js.html that gives 8.9 in Firefox. Lua 5.1 gives 14.68.

SciMark code runs, but I get this error:
Moonshine Error: Attempt to call non-function
@scimark.lua [./null:424]

[paulcuth]

Hi Stuta,

Sorry for the delay, I've been away.

It turns out that string.format() was having issues when it was passed a format string containing new lines. This has now been fixed.

The error that you were seeing was a result of the io library not being fully implemented. I've now padded it out with placeholder methods and io.flush() should no longer error. There are future plans for implementing a filesystem layer that stores data in localStorage, but that is low on our priority list at the moment.

As for the score, it is disappointing but, as I said earlier, it's not a bottleneck in the performance of our games and we're hoping to improve speeds in the future.

Thanks for continuing to give feedback, it's much appreciated and is improving the project all the time.