search

bem-archive / bem-bl-xjst

XJST Compiler for bem-bl. DEPRECATED. USE https://github.com/bem/bem-xjst
1 stars 0 forks source link

i-bem: 30% speed improvement by better escaping #6

Closed indutny closed 9 years ago

indutny commented 9 years ago

Please merge and release! ;)

tadatuta commented 9 years ago

cc @arikon

arikon commented 9 years ago

@indutny Where to read about $$registerExtensions()?

arikon commented 9 years ago

@indutny BTW, :+1:

mishanga commented 9 years ago

Неужели эскейпинг и правда дает заметный прирост? Надо прикрутить в BH.

indutny commented 9 years ago

@mishanga стрельбы покажут :)

@arikon this is a new xjst thing :) Basically it adds all properties that are used in local()/apply() statements to the context, thus avoiding hidden class creation.

tadatuta commented 9 years ago

all tests in islcomponents passed

mishanga commented 9 years ago

@andrewblond we need this update in enb-xjst: https://github.com/enb-bem/enb-xjst/blob/master/package.json#L30

arikon commented 9 years ago

@mishanga It has errors, we are trying to investigate

indutny commented 9 years ago

@arikon have you tried 2.1.1? 2.1.0 might be broken on some templates, indeed.