Opened 3 years ago

Closed 3 years ago

#236 closed ожидается проверка (задача сдана)

HW #1 (BMP) (pavlenko.daniil)

Reported by: Daniil Pavlenko Owned by: Дмитрий Лапшин (lapshin)
Component: HW #1 (BMP) Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Daniil Pavlenko
Type: ожидается проверка → ожидаются исправления

Корректность:

  1. Один из заголовков про размер пишется неправильно.

12/20.

Бонус вроде норм.

Стиль:

  1. Ворнинги.
  2. Давай всё же использовать типы гарантированного размера. И имена в одном стиле.
  3. Многовато волшебных чисел без имён.
  4. Наверное было бы полезно напечатать на экран что за ошибка произошла, если уж произошла. Кстати было бы не лишнее в случае ошибки не потерять память картинки.

6/10.

comment:2 Changed 3 years ago by Daniil Pavlenko

Owner: changed from Daniil Pavlenko to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

Вроде пофиксил заголовки
Стиль:
1) Исправил
2) Исправил (все переписал в стиле CamelCase)
3) Исправил
4) Исправил ошибку с освобождением памяти. Вроде, выводил ошибки на экран (bmp.c : 29, 77, 99 строчки), добавил еще обработку ошибки при выделении памяти

comment:3 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Daniil Pavlenko
Type: ожидается проверка → ожидаются исправления

Variably modified trash at file scope!

А жаль, хорошо бы работало.

Корретктность: 15 за CE.

Бонус: зачёт

Стиль: да, ок, 10.

comment:4 Changed 3 years ago by Daniil Pavlenko

Owner: changed from Daniil Pavlenko to Дмитрий Лапшин (lapshin)
Type: ожидаются исправления → ожидается проверка
Version: 2.0 → 3.0

Вроде исправил (вынес константу как define).
(Хотя странно, что локально все компилилось...)

comment:5 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed

20 10 10

Note: See TracTickets for help on using tickets.