Hw01 - Githubissues

ArtyomLobanov commented 5 years ago

Также ""=text даёт весьма странный результат

RamSaw commented 5 years ago

Также хотелось бы, чтобы работала такая последовательность команд:
a=ec
b=ho
$a$b text
А вот такая НЕ работала:
echo text |

Fixed. На все замечания добавлены тесты.

RamSaw commented 5 years ago

@ArtyomLobanov вроде все замечания пофиксил. Я для удобства и более правильной имитации баша добавил класс Pipeline, который имитирует pipe и ведет себя по сути, как файл.

RamSaw commented 5 years ago

Также ""=text даёт весьма странный результат

теперь выдается ошибка, как в баше.

RamSaw commented 5 years ago

@ArtyomLobanov пофиксил замечания. Вообще на практики Юрий говорил, что копировать полностью bash не надо. Я понял так, что достаточно поддержать кавычки одинарные и двойные, в том смысле, что в них работала бы правильно подстановка...Не знаю, надо ли предусматривать всякие экзотические случаи баша (ну не то чтобы экзотические, но я, к примеру, могу потребовать, чтобы мой интерпретатор требовал ставить пробелы между "|"). Если надо, скажите до какой степени, я попридумываю тестов, чтобы их покрыть и сделаю lexer так, чтобы он правильно интерпретировал все требуемые случаи. Просто довольно трудно что-то писать, когда грамматика слишком неопределена.

ArtyomLobanov commented 5 years ago

Если надо, скажите до какой степени, я попридумываю тестов, чтобы их покрыть и сделаю lexer так, чтобы он правильно интерпретировал все требуемые случаи.

Текущая версия меня вполне устраивает. За настройку travis для сборки под windows респект. Передаю слово @yurii-litvinov

RamSaw / SoftwareDesign

Hw01 #1