#2446: V3: Event System
--------------------------+-------------------------------------------------
Reporter: fredck | Owner: fredck
Type: New Feature | Status: assigned
Priority: Normal | Milestone: CKEditor 3.0
Component: General | Version:
Resolution: | Keywords: Confirmed V3ProtoBase Review+
--------------------------+-------------------------------------------------
Changes (by martinkou):
* keywords: Confirmed V3ProtoBase Review? => Confirmed V3ProtoBase
Review+
Comment:
The most critical piece of code here is CKEDITOR.event::fire(), as it is
expected that it will be called a lot. I used the following code snippet
to test its performance:
{{{
a = new CKEDITOR.event();
for(var i=0;i<100000;i++)
a.fire('nosuchevent', '');
}}}
The previous prototype code created two closures called stopEvent and
cancelEvent for each time fire() is called... which doesn't seem very
efficient. I've moved the creation of the two closures out in [2349] to
improve performance. I've got a nearly 3x improvement in performance with
the new code according to Firebug's profiler with the above code snippet.
--
Ticket URL: <http://dev.fckeditor.net/ticket/2446#comment:2>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
