length expando overwrite

[dbjdbj]

We know that length expando is overwritten by certain browsers, in certain situations. Maybe a new utility function should be added to the core.js ?

safeLength : function ( obj ) {

  if ( jQuery.isArray(obj)) return obj.length ;
  
  var j = 0; while ( obj[j++] ) ; return j-1 ;

}

This could/should be used from merge(), grep(), map() and each()

--DBJ

[dbjdbj]

I am thinking ... maybe I have jumped on this one too eraly? I think that each() will solve safely "length expando overwrite" because it does objects (ie non-arrays) with for/in loop anyway? that is it does not use length. So maybe each() as it is, does the job? the solution is to make grep(), merge(), map() etc, to use each().

safeLength() might stay, but I do not see where it could be used inside jQuery ? There must be some use for it ?

--DBJ

[dmethvin]

This seems more appropriate for the jQuery-dev group since it isn't reporting a bug.

[dbjdbj]

I admit I "made it" into bug to gain attention. Please look into my fork on GITHUB (user: DBJDBJ ) I managed to streamline each() using the above and to streamline merge(), grep(), makeArray() to use each(). I have done testing. This needs to be tested/seen by someone from the core team. Looks ok and obvious, but is a non-trivial change in the core.

Thanks: Dusan

[john]

This isn't really something that we need in core. We just do looping in a particular way to avoid the problem outright.