Opened 5 years ago

Closed 5 years ago

#61 closed ожидаются исправления (задача сдана)

WW #1 kreslavskiy.kirill

Reported by: Kreslavski Kirill Owned by: ldvsoft
Component: WW_make Version:
Keywords: Cc:

Description


Change History (4)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Kreslavski Kirill
Summary: WW #1 → WW #1 kreslavskiy.kirill
Type: ожидается проверка → ожидаются исправления

Привет!

Да, заслал всё правильно, есть парочка недостатков:

  1. Объектные файлы должны лежать в подпапке bin/. Саму папку надо или положить в репозиторий (пустую), или дописать правила для её создания, что я расскажу завтра.
  2. В коде у тебя разные отступы. Пожалуйста, соблюдай стиль и придерживайся какого-то одного размера отступов. Разумные рекомендации — 4 пробела или символ табуляции, но я приму любую консистентную. Причём где-то таб, где-то 4/8 пробелов не очень хорошо.
  3. Соблюдай, пожалуйста, стиль и в других конструкциях. Например, у тебя в util.c не хватает пробелов: if (condition) {.
  4. Заголовочным файлам не хватает guards. Здесь не то чтобы критично, но давай добавишь, не обязательно соблюдать стандарт ;)

comment:2 Changed 5 years ago by Kreslavski Kirill

Owner: changed from Kreslavski Kirill to ldvsoft
Type: ожидаются исправления → ожидается проверка

Я всё исправил.

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

Type: ожидается проверка → ожидаются исправления
Version: 1.0

А на меня назначить?

Include guards?

Следи за стилем.

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

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

Дедлайн вышел.

Note: See TracTickets for help on using tickets.