SpriteList still drawing elements after they're removed about jaws

[ippa]

good find! .. actually your demo works just fine in both latest chrome and ie9 but fails (failing as in list[0] contains element1) in firefox 9 (and I assume earlier versions as well).

But of course it needs to work in firefox as well. Do you think some simpler patch is possible to solve this issue, maybe setting list[x] to undefined manually where it matters? Probably a bad solution though.

A patch from someone who spent time digging into the issue sounds very good in my ears :). I would most certainly accept it.. some tests for this would kick ass as well. I wouldn't mind having the internal array be called sprites.. cause what does a SpriteList contain if not sprites :). Also let's try not to break any documented APIs.. but I don't think that should be so hard.

[dmitrizagidulin]

Heh, that was my first thought too -- calling 'delete' so that list[x] is set to undefined.. and then wrapping the draw() invocation on each sprite in a if(this[i]) { this[i].draw() }.

(Also, I was /wondering/ if this might be a browser-specific issue, too! I see)

But.. I mean, if the language protests against subclassing Array, then fine, let's not. Let me get a patch together (with unit tests, definitely), see what you think.

[dmitrizagidulin]

I copy on the internal array being called 'sprites', btw. :)

[dmitrizagidulin]

Oh hey, question. Do all the current unit tests for jaws pass for you, for all browsers? (I'm working on adding some, to deal with the array issue right now).
All of the current ones pass for me in Firefox 9, but I'm getting failures in Chrome 16 and IE 9 (on windows)

[ippa]

works 100% for in me in chrome.. in ie9 and ff there was some kind of race condition producing a lot of errors, I think I fixed it now.

I only get one failing test in ie9/ff now, and that's "audio loaded".. not sure why the .wav-file isn't loaded correctly just yet.

[dmitrizagidulin]

Question. In sprite_list.js, in the constructor:

jaws.SpriteList = function SpriteList(options) {
  if( !(this instanceof arguments.callee) ) return new arguments.callee( options );

What is that second line (re arguments/callee etc) for? I think I understand what it's saying, but what situation does it come useful in?

[ippa]

it makes both sprite_list = new SpriteList() and sprite_list = SpriteList() work

.. basically it will make the constructor work without "new". arguments.callee is just a more general way of writing SpriteList.

[dmitrizagidulin]

One more - what's your policy on tabs vs spaces, in contributed code? (tab chars ok, or do you want '2 spaces' tabs, or what?)

[ippa]

2 spaces