Инструменты авторинга при некорректном регексе
Issue #194
closed
Originally reported on Google Code with ID 194
В отличие от матчера, инструменты авторинга должны работать при некорректном регексе.
Дерево может отображать почти любые ошибки - как и описание. Граф - некоторые (это
должно решаться аццептингом узла ошибки с вниманием к ее типу). Управлять возможность
отображения дерева/графа/описания должен аццептинг, а не наличие ошибок.
Это помогает юзерам находить и исправлять ошибки. Поэтому с последним коммитом, отменяющим
построение конкретного дерева при ошибках я не согласен.
Давайте разберемся и наладим работу инструментов авторинга в случае некорректного регекса.
С алертами надо разбираться - система узлов дерева с ошибками и вводилась для того,
чтобы иметь возможность отобразить регекс при наличии ошибок.
P.S. Строки типа
dotscript = 'digraph { "Ooops! Your regex contains errors, so I can\'t build the explaining
graph!" [color=white]; }';
вообще мне странны. Вроде все участники давно в курсе, что любая строка, выводимая
пользователем, должна идти через файл языка чтобы переводиться...
Reported by oasychev
on 2013-04-21 21:23:01
Comments (6)
-
-
reporter На самом деле, если вы подумаете, то поймете что дерево и описание действительно могут отобразить все или почти все. Если чего-то нет ни в аццептинге, ни в инструментах - надо посмотреть, куда его добавить - может и в дерево/описание, а не откидывать. Вы начали, поэтому пока владелец. Вернуть строки мало, надо изменить функции сообщения об ошибке в регексе на функцию вывода сообщений о непрохождении аццептинга например... Ну и сделать анализ алертов, чтобы знать, кому их исправлять.
Reported by
oasychev
on 2013-04-21 21:54:15 -
Вернул построение dst как было; перенес проверку на якорение в матчер под условием что ошибок нет - алертов тоже нет, все хорошо. Поскольку заговорили про акцептинг, Терехову также нужно переопределить всякие функции типа получения имени компонента и т. д., у остальных вроде нормально.
Reported by
vostreltsov
on 2013-04-21 22:20:46 -
reporter Валерий, по этому issue есть еще проблемы? У Терехова вполне может быть аццептинг всего на свете - в дереве не проблема отобразить все...
Reported by
oasychev
on 2013-07-02 14:21:46 -
reporter Валерий, вы не ответили на последний комментарий.
Reported by
oasychev
on 2013-07-12 14:45:21 - Status changed:Fixed
-
Тут вроде всё нормально.
Reported by
vostreltsov
on 2013-07-13 05:20:41 - Status changed:Done
- Log in to comment
Reported by
vostreltsov
on 2013-04-21 21:45:04