problems with increments in arrays

[espruino-discuss]

Posted at 2014-01-25 by Kim

The following code causes an infinite loop and Out of Memory warnings:

  var contents = [1,1,1,2,1,3,2,1,3,1,3,2,4,1,5,1,6,1,6,2,6,3];
  var i = 0;
  for (i = 0; i < contents.length; i = i + 2) {
    print(contents[i]);
    print(contents[i+1]);
  }

The problem has to do with print(contents[i+1]);so I assume the issue is in the use of i+1at that spot.

The following modification exposes the same, bad behaviour:

    print(contents[i]);
    j = i + 1;
    print(contents[j]);

[gfwilliams]

Posted at 2014-01-25 by @gfwilliams

Thanks. I'll look into this after the weekend. It could actually be a regression in the latest version of the software as I had tried to speed up array accesses.

[gfwilliams]

Posted at 2014-01-27 by @gfwilliams

I just tried both of these and they work fine for me. Either pasted into the left or right hand sides of the Web IDE.

Maybe someone else can confirm whether they see the problem as well?

[espruino-discuss]

Posted at 2014-01-27 by Kim

I'm not seeing it on all my boards ... Just tried it with another one and there are no issues. I still have the original code lying around so will try that again this evening on the original board (no time now, and code used the screen so need to do some wiring up).

[espruino-discuss]

Posted at 2014-01-27 by Frida

It works both on my discovery and my espruino's.

[espruino-discuss]

Posted at 2014-01-27 by Kim

This will probably have been myself goofing up. I have had issues with the screen since day one, where sometimes it works flawlessly and other times, after reconnecting, I cannot get anything drawn on there. Might just be faulty wiring or something that is the cause (no idea how to check that though), but it definitely sounds like it isn't the software.

[gfwilliams]

Posted at 2014-01-27 by @gfwilliams

It's possible it's a missing character - the problem should have gone away with changes in the last few months, but I guess it is possible that occasionally characters will still get lost.