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

Поддержка JS RE #2

Closed
wants to merge 2 commits into from
Closed

Поддержка JS RE #2

wants to merge 2 commits into from

Conversation

afelix
Copy link

@afelix afelix commented Jun 29, 2011

Добавлена поддержка JS RE вида /../gim в JS-транслятор.

@veged
Copy link
Owner

veged commented Jun 29, 2011

Зачем поддержка отдельно диапазонов, если всёравно после парсинга их не остаётся?
Разделение на первый символ и хвост покрывает лишь небольшое количество неправильных регекспов.
Предлагаю сделать простое правило похожее на строки (fromTo и ескейпинг), только плюс поддержка флагов.

@afelix
Copy link
Author

afelix commented Jun 29, 2011

Разделение на первый символ и хвост отделяет комментарии от регекспов, у него нет задачи парсить что-то неправильное: ECMA-262-5 // A.1 Lexical Grammar, стр. 214.

Давай так: вот это вроде (отсутствие тестов бодрит) работающий патч потенциально полезной фичи, сделанный по грамматике JS, затем немного упрощённый. По грамматике (а не из головы) он для того, чтобы следующий желающий в этом компиляторе что-то сделать мог считать ECMA-262-5 документацией к коду, а не бесплатным приложением к фантазиям автора. В то же время отсутствие этой фичи блокером не является, мы всегда можем клеить файлы вне компилятора. На него я потратил несколько часов. На данный момент больше тратить не хочу.

@veged
Copy link
Owner

veged commented Jun 29, 2011

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

пуллреквест -- это взаимодействие людей

@afelix
Copy link
Author

afelix commented Jun 30, 2011

О безоговорочном речи нет. Я тебя не заставляю мёржить фикс — как обычно, выбор за тобой.
Починка уже произошла. Т.к. она тебе не нравится, ты её не принимай, вот и всё.

@afelix
Copy link
Author

afelix commented Jun 30, 2011

PR закрываю. Оригинал Ometa/JS вдруг снова ожил, форкнул его, отправил PR автору.

@afelix afelix closed this Jun 30, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants