HW #1

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

Корректность -- 20/20
Тесты с первого раза все прошли, молодец. Можешь делать бонус.

Стиль -- 8/10

1. Зачем null_buffer выделять на куче? Он маленький и фиксированного размера -- прекрасно будет себя чувствовать на стеке.
2. Лучше memset:

       for (size_t i = 0; i < 4; i++)
           null_buffer[i] = 0;
   

3. В init_pixel_array утекут ресурсы, если вдруг первая аллокация пройдет успешно, а вторая нет. Если ты внутри функции решил не падать на ассерте, а возвращать NULL или какую-то ошибку в случае если что-то пошло не так, то твоя функция должна освободить все ресурсы, которые она успела выделить.

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

Стиль 9/10, первое и второе замечание не пофикшены.

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

Насчет бонуса -- расшифровывается некорректно, какой-то левый символ дописывается в конце сообщения и перевод строки не добавляется.
5/10

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

Всё хорошо, только зачем null_buffer делать указателем на указатель?
Когда ты пишешь char null_buffer[4] -- это буфер на стеке размера 4 байта и у переменной null_buffer тип char*, а когда ты написал char* null_buffer, то это буфер из четырех указателей на char, размер у него 4*sizeof(char*), а переменная null_buffer имеет типа char**.

Бонус работает
40/40