подожди, я имел в виду с другой реализацией

var reduceValue = options.reduceInit;
for( var i=0, j=0; i<els.length; i++ ) { 
...
        if( options.reduce ) 
            reduceValue = options.reduce.call( options.scope, reduceValue, el, j );

и реализация стала проще. Можно добавить еще значение по умолчанию 0/пустая строка/whatever

а размер зачем? А как ты решения принимаешь? На глаз?Я пока рассчитываю, что, по-крайней мере в большинстве случаев, не имеет смысла жертвовать читабельностью ради производительности. В частности, если не пытаться реализовать поиск по сферическим селекторам в вакууме :)

Ну и я не говорил, что нужно воспринимать тот код, как готовые решения. Можно, например, просто парсинг селекторов вынести

Все-таки лучше оставить стандартное для reduce поведение, чтобы не путаться.

Хм, ну чтоб меньше был.

Разумеется всё сугубо субъективно и по настроению. Наутро встану -- перепишу :) Обсудить-то мне это не с кем :)

вот это мне и не нравиться. Ты ставишь читабельность на третье место, не зная, что это тебе дает на практике

Ну для себя-то у меня всё обосновано, а больше мне некому обосновывать. Если хочешь, можешь форк сделать, а там и смержим что-нибудь :)

Это не значит, что она не важна в принципе. Просто если из двух вариантов читабельный сильно больше места занимает или работает неэффективно, то я выберу нечитабельный вариант. Но это должна быть большая разница в размерах, байты я, конечно, не считаю. В прикладном коде всё наоборот, на первом месте читабельность, т.к. его постоянно переписывать приходится.

Это мне даёт маленькую функцию с огромными возможностями по выборке, которую я могу подключить без опаски притянуть за ней огромный фреймворк. Поэтому размер и эффективность на первом месте -- требования к функции меняться не будут, а значит и переписываться она, в идеале, не будет, только правка багов. Но, разумеется, код должен быть максимально читабелен.

да я уже немного попробовал, пока что ты смержил только баг ;) Ну понятно, в общем-то, проиритеты разные. А вообще, хочу позже что-нибудь выложить. Пока рано

Метод clone в http://alljs.ru/crossjs/source/lang/Array.js правильно работает только с одномерными массивами. Так и задумано?

var array1 = [[]];
var array2 = array1.slice(0);
alert([
    array1 === array2,
    array1[0] === array2[0]
]);

Да я, честно говоря, и не задумывался об этом ))

В файле Element.js в функции M.removeClass используется регэксп:

new RegExp('($|\\s)' + cl + '(?=\\s|$)', 'g')

Вероятно, в первой скобке вместо доллара должна быть крышка (^)?

stopkran,
спасибо, поправил. Неприятный баг.

А вот и небольшая критика M.addClass: http://ir2.ru/show-hide-toggle.aspx (там больше критики obj.style.display = "none", поэтому отдельная статья)