OnionGrief / Chipollino

преобразования регулярных выражений и конечных автоматов
Other
20 stars 4 forks source link

Colorize #269

Open TonitaN opened 1 year ago

TonitaN commented 1 year ago
TonitaN commented 12 months ago

Возникает вопрос, где хранить выделенные цвета для состояний автомата (например, номер цвета, соответствующий ловушке), и вообще, не нужно ли перенести методы разметки на уровень абстракции выше.

xendalm commented 12 months ago

А может в отдельный класс вынести раскрашиватель, где он будет иметь все, что нужно. Чтобы не встраивать его в существующие объекты. Ещё не успел отследить, что изменилось в раскраске на текущий момент. Но методы типа mark_all_transitions, наверное, получится вынести в friend класс

xendalm commented 12 months ago

Ну по сути нейпспейс, а не класс получится, если нам нечего хранить

TonitaN commented 12 months ago

А поздно, уже класс)

TonitaN commented 12 months ago

Раскраска SemDet требует переноса последних изменений в (ныне забагованном в main) SemDet. Не думаю, что будет хорошей идеей откладывать первый PR по Colorize до полного внесения изменений в частях Эдгара, тем более, их логично делать в другой ветке.