Opened 5 years ago

Closed 5 years ago

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

WW #1

Reported by: kurnosova.polina Owned by: Vasily Alferov
Component: WW_make Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 5 years ago by Vasily Alferov

Component: HW #1 (BMP)WW_make
Type: ожидается проверкаожидаются исправления

Здорово, мне нравится. Нужно поправить только несколько замечаний.

  • На этапе линковки ты передаёшь src/main.c вместо bin/main.o.
  • Почему bin/algorithm.o зависит от bin/util.o и bin/io.o?
  • io.h — почему прагма закоменчена? Кстати, там опечатка в прагме: prargma вместо pragma.
  • Давай не будем жалеть пустых строк, я почти уверен, что такое есть в почти любом стайлгайде. Нужно добавить после прагмы, а также между инклюдами и функциями.
  • Пробелы при передаче аргументов где-то есть, а где-то нет.
  • Отступы: везде табы, но в 5 строчке src/util.c табуляция, а потом два пробела. Давай лучше две табуляции?
  • Первой строчкой файла file.c принято подключать file.h, если он есть. Это сделано для того, чтобы на этапе компиляции верифицировать, что у тебя в исходниках написано то же, что обещано в хедере.
  • algorithm.c — в шестой строке восемь пробелов вместо табов.

comment:2 Changed 5 years ago by kurnosova.polina

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

comment:3 Changed 5 years ago by Vasily Alferov

Resolution: задача сдана
Status: assignedclosed

Зачтено.

На будущее: ну, издеваться надо мной так не надо :)
Пробелов можно и поменьше. Они точно лишние после открывающих круглых скобок и перед закрывающими. Хотя кое-где я видел и твой вариант, в целом, он менее распространён.
Почитай первые две с половиной страницы этого документа, там есть какие-то примеры.

Note: See TracTickets for help on using tickets.