Сообщение от Riim
|
|
1. в commonHandle переменная handlers попадает в window. Нужно var добавить.
|
Ага, почистил.
|
Цитата:
|
|
2. непонятно зачем в elem.handle проверка (typeof Event !== "undefined")
|
В статье про это сказано - это для фильтрации редкой ошибки с unload страницы.
|
Цитата:
|
|
3. содержимое commonHandle вынесено в отдельную функцию. Вероятно для увеличения производительности Event.add, но это делается в ущерб производительности elem.handle, которая имеет заметно больший приоритет.
|
Это чтобы не создавать кучу лишних функций commonHandle и не расходовать тем самым память. Конечно, можно ее заинлайнить.
|
Цитата:
|
4. конструкции типа:
...
Там еще множество разных оптимизаций можно применить, но все они в ущерб читабельности. А эта вроде нет.
|
Да, приятное исправление, сделал его. Особо серьезных оптимизаций там даже в ущерб читабельности не сделаешь имхо.