Козлов Василий Сергеевич P3215 Лаб 4
- Owner
@vaskozlov посмотрел код, два комментария:
- для структуры
block_search_result
стоит сделать конструктор и не создавать её явно чтобы избежать потенциальных ошибок; - стоит добавить проверку на инициализацию кучи во всех внешних интерфейсах и отслеживать это. Например в функции
heap_term
стоит проверить что куча была инициализирована и сбросить этот флаг по зваершению, в других аналогично. Можно использовать имеющуюсяHEAP_ACTUAL_START
как флаг инициализации кучи, только надо будет сделать её не константой
- для структуры
@andrew исправил. У меня не получилось добавить проверку на инициализацию кучи при вызове _free и heap_init, потому что тесты не всегда корректно соблюдают внешний интерфейс. Может быть конечно стоило написать свои тесты, но я как-то не понимаю, как качественно протестировать аллокатор)
P.S. динамически устанавливать начало кучи у меня тоже не получилось опять же из-за тестов
Edited by Vasiliy Kozlov- Owner
@vaskozlov да, тесты дейсивтиельно очень жестко регламентируют реализацию и дают мало свободы, можно было конечно немного изменить тесты добавив в них принудительную инициализацию, подумаю над этим для следующего года
лаба принята
closed
- You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
Please register or sign in to reply
Compare and