Am 12.11.2012 13:12, schrieb Martin Wittemann:
> Hey,
>
>> in my qx.event.Manager.js#654
>>
>> the entryList is sequentially looped:
>>
>>  for (var i=0, l=entryList.length; i<l; i++)
>>  {
>>    entry = entryList[i];
>>    if (entry.handler === listener && entry.context === self)
>>    ...
>>
>> Now the debugger tells me that in my case
>>
>>   entryList.length == 53033 (!!!many items!!!)
>>
>> so everything gets very slow.
>>
>> 1) Does this show some problem in my code or are all qooxdoo events inside 
>> entryList
>>  and the huge number is normal?
> Well, it depends on the application size. If its a huge application, a number 
> of 53k listeners is possible, especially when using data binding. So thats 
> nothing I would worry about.
I have a qx.ui.table.model.Simple which has many CellEditors, probably
it is related to them, but currently only filled with a few rows.
In future some 12'000 rows will be entered.

>> 2) Shouldn't the access to entryList be via a hashtable?
> And what would the key be? Listener and context could be two complex 
> widgets...
The hash value or a compound key string of both above or, a hashtable of the 
first which contains a
hashtable of the second.

>
> Did you profile how much this is impacting the overall performance? If its 
> enough, we might consider a bug report for that.
The current Firefox stops processing and a popup tells if we want to stop the 
app
because of slow execution. When I enter debugger then it displays above loop.
On a 8-core 3.4GHz Linux.

Thank you
Marcel


------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to