Ошибки в графе и дереве на спец символах

Issue #220 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 220

Полюбуйтесь на
\s*(?:printf|puts)\s*\(\s*"how do they \\"print double-quotes\\""\s*\)\s*;\s*
на ваших инструментах

Сильнее всего пострадал граф- судя по месту начала ошибок, у него проблема с двойной
кавычкой просто встретившейся в регексе.
В дереве  же кавычки нормально, но проблема на эскейпенных круглых скобках.

Reported by oasychev on 2013-09-05 12:15:42

Comments (11)

  1. Oleg Sychev reporter
    Нужно сделать тест на все символы, которые могут быть служебным в DOT и HTML...
    И юнит-тестом его...
    

    Reported by oasychev on 2013-09-05 12:18:16

  2. Former user Account Deleted
    Исправлена ошибка с двойной кавычкой, приводящая к краху.
    

    Reported by ZluMYO on 2013-09-19 11:06:16 - Status changed: InProgress

  3. Oleg Sychev reporter
    В дереве сейчас кавычка (двойная) выглядит криво: во-первых она смещена к левому краю
    своего прямоугольника, во вторых тултип пишет character "
    

    Reported by oasychev on 2013-09-25 19:35:49

  4. Valeriy Streltsov
    Еще граф почему-то стал тупить на русских символах
    

    Reported by vostreltsov on 2013-09-26 08:50:34

  5. Oleg Sychev reporter
    Прежние проблемы пофикшены.
    У меня вроде русские символы сейчас нормально в графе отображаются.
    
    Зато выяснилось, что и дерево и граф имеют проблемы со строками, буквально отображающими
    html-сущности. Если в регексе ввести 
    <
    то в графе будет 
    <
    вместо этого.
    А если 
    [<]
    то аналогичные проблемы поимеет и дерево.
    
    При регексе
    []
    граф и дерево вообще накрываются...
    

    Reported by oasychev on 2013-10-01 18:34:14

  6. Oleg Sychev reporter
    У графа есть некоторые проблемы с выделением синим цветом на сложных чарсетах - может
    ну его? Налазит очень некрасиво, а спутать там вроде нельзя ничего...
    

    Reported by oasychev on 2013-10-01 18:38:58

  7. Oleg Sychev reporter
    Иванов - еще вам надо поправить отображение ошибок принятия в графе. Вы когда писали
    is_preg_node_acceptable читали к ней комментарий внимательно? Что она должна вернуть
    если не принимает узел? Не просто false - а строку для пользователя (т.е. полученную
    через get_string), что именно не принимает... Аналогично и метод accept себя ведет.
    Часто можно воспользоваться методами типа tohr в узлах.
    
    А то у вас сейчас в случае ошибок обрезанное сообщение выдается, что именно не принимается.
    Исправьте и проверьте для каждого типа узла что сообщение нормальное!
    

    Reported by oasychev on 2013-10-04 13:23:30

  8. Former user Account Deleted
    Инструменты авторинга не работают на следующих тестах:
    1) (*COMMIT)
    2) aaa[3-5!]\R
    
    пока что нашёл только эти...
    

    Reported by grvlter on 2013-10-06 16:10:10

  9. Valeriy Streltsov
    1) пофиксил - проблема была в дескрипшене
    2) \R не поддерживается даже на уровне лексера - Наравне с \K, \X. Надо либо доделать
    их, либо try-catch в хэндлере поставить и accepting error
    

    Reported by vostreltsov on 2013-10-07 08:57:24

  10. Oleg Sychev reporter
    [<] в дереве остается проблемой, в графе нет - Терехов, посмотрите на dot код и возьмите
    решение. Что-то у нас ничего не движется....
    

    Reported by oasychev on 2013-10-18 18:07:38

  11. Log in to comment