-
Notifications
You must be signed in to change notification settings - Fork 25
Finished x86 asm compiler #1
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
Conversation
|
Убедитесь в том, что PR автоматически мержится, и делайте его в ветку dev. Убедитесь в том, что вы не модифицируете файлы в корневой директории. Перетяните существующую структуру себе в директорию. Существенных замечаний по текущей реализации нет, все проблемы "всплывут" во время выполнения следующих домашних заданий. |
|
@kajigor я правильно понимаю, что это замечание уже по самому последнему заданию? (Я, к сожалению, не был на последней лекции). Т.е. нужно привести код к модульной структуре (которая сейчас в мастере лежит)? |
|
@SammyVimes Это был комментарий по прошлому заданию. В новом там надо убедиться, что стек в вашем решении ведет себя правильно (скорее всего, нет) |
|
@kajigor а код можно старый использовать или надо из мастера забрать? |
|
@SammyVimes А как вам угодно. Лучше, конечно, свой код привести в соответствие с текущей структурой в мастере. |
Данилов Семён,
группа 2304,
senya.danilov@gmail.com