Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Странное поведение в версии 0.4 #33

Open
maksimr opened this issue Sep 16, 2013 · 3 comments
Open

Странное поведение в версии 0.4 #33

maksimr opened this issue Sep 16, 2013 · 3 comments
Labels

Comments

@maksimr
Copy link

maksimr commented Sep 16, 2013

Наткнулся на запись вида:

  MyClass('ClassName', {/*borschik:include:file.js*/});

в версии 0.3 - это не работает. Работает в версии 0.4.
Сейчас в ветке rc-0.4 это поведение описано, но
мне кажется это излишне и вводит в замешательство(очень не очевидно в силу своей странности).

{/*borschik:include:file.js*/} и [/*borschik:include:file.js*/] просто заменить на запись вида:

  MyClass('ClassName', "borschik:include:file.js");
  MyClass('ClassName', ["borschik:include:file.js"]);
@alexeyten
Copy link
Contributor

А что странного?
Ноги растут из #16 и лично мне это кажется логичным расширением существующего синтаксиса.
Синтаксис "borschik:include:file.js" уже занят.

@maksimr
Copy link
Author

maksimr commented Sep 16, 2013

Действительно эффект разный...

Почему не сделать просто:

 require('file.js');
 borschik('file.js');

Если хотите гибкости, позвольте самому настроить название функции.

Чем сейчас плох ваш синтаксис тем что:

  1. Он не понятен
  2. Он уже с ошибками
Jane.Handler('test', {
    /* borschik:include:test.params.js */
});
/*
Error
    at new JS_Parse_Error (/tmp/b/node_modules/borschik/node_modules/uglify-js/lib/parse.js:196:18)
*/

Писать слитно фигурные скобочки ломает автоматическое форматирование.

@doochik
Copy link
Contributor

doochik commented Oct 3, 2013

Функции мы уже проходили. И главная проблема в коде с комментариями

/*
borschik.include('file.js') // а в file.js тоже есть /**/ и после ракрытия получится невалидный код
*/

@doochik doochik closed this as completed Oct 3, 2013
@doochik doochik reopened this Oct 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants