WW #6 Bobrov

[Святослав Власов]

Корректность -- 17/20

1. Такой тщательной обработки нештатных ситуаций с пробросом ошибок наверх не требовалось, достаточно было просто падать на assertе, но раз уж ты это делаешь, то делай хорошо. Ты в массе случаев не проверяешь возвращаемое значение своих же функций, позволяешь в нештатных ситуациях ресурсам течь и программе падать в UB.

Бонус -- 10/10

Стиль -- 5/10

1. Длинные строки (>80 символов) нужно подрезать
2. Проверять указатель на NULL перед вызовом free избыточно. Вызов free(NULL); не является ошибкой и просто ничего не делает.
3. Создай структуры для каждого хедера и читай шапки из файла в них. У нас, конечно, низкоуровневое программирование, но не настолько, чтобы руками байтовые смещения высчитывать. Вспомни что такое #pragma pack (или узнай, если тебя на позапрошлой практике не было) и читай хедера в структуры, а не в массив байтов.
4. Вместо побайтового обнуления массивов используй memset.