[Laszlo-dev] For Review: Change 20081021-Philip-6. Summary: Iterate over arrays with for "for (var a=0; a<b.length; a++)"

[André Bargull]

On 10/21/2008 9:11 PM, P T Withington wrote:
> The profiler is using a terrible trick of recording a clock tick and a 
> function name 'efficiently' by using the clock tick as a key and the 
> function as a value.  It might be just as, or more efficient, to simply 
> record both in an array where the even index is the clock tick and the 
> odd index is the function.

Sounds reasonable to choose that approach.

> 
> This bug wouldn't be relevant if the buffers were Dictionary's.  Max 
> made them Array's to work around IE's poor GC performance with objects.

Oh yes, I remember that article about IE's GC strategy. It makes my head 
hurt... ;-)


> 
> On 2008-10-21, at 15:04EDT, André Bargull wrote:
> 
>> IMO it's bad to use l because of 1, I'd suggest to use "len" instead.
>>
>> LzProfile.lzs still needs to be fixed (it iterates over its buffers 
>> with a for-in loop). There was something special about the profiler, 
>> you've explained it to me once, but I don't know whether it's still 
>> valid in respect to your recent changes.
>>