Slava Oliyanchuk

@tadatuta Updated according to your idea. Benchmark result ~ +2%… cc @@zxqfox

@tadatuta > How many wrap/extend calls are there in bench templates? 11. (and ~450 `block()` subpredicates). How many times `wrap()` was matched I don’t know. > 2% slower then previous...

No. can you describe a reason why it’s needed for you? On Mon, 7 Nov 2016 at 12:29, Andrew Abramov [email protected] wrote: > Any news? > > — > You...

The same issue with `extend()`: ```js $ cat extend.js const bemhtml = require('./').bemhtml; const tmpl = bemhtml.compile(function() { block('b1').extend()({ 'ctx.content': 42 }); }); const bemjson = { block: 'b1' };...

@tadatuta I see only one solution: copy bemjson to a new object in first line of `bemhtml.apply()`. Is it OK?

Я про эту идею уже думал, она мне очень нравится. Вечером попробую изменить профит от такой экономии. И, конечно, это мажорное изменение.

Сравниваю ревизии: ``` rev1: 'd830717eed5b9c4527c317ec4d71ed8be3eb017b', // после оптимизации rev2: '8083faba3dd01657d88b3def5a796c2099f369c2', // до оптимизации ``` На 2000 разных BEMJSON-ах. Довольно больших, взятых с реальных проектов. Результат не очень разный до и...

Разумеется. Я ни в коем случае не спорю в этом вопросе. Результаты просто говорят о том, как часто в шаблонах, которые я использовал для теста, используется `bem:false` (18 раз) и...

В итоге я не очень согласен с формулировкой заголовка таска. Так как не считаю, что производительность именно страдает. Но улучшить безусловно стоит.

@vithar nope.