Opened 5 years ago

Closed 5 years ago

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

WW #1 shein.andrey

Reported by: Андрей Шеин Owned by: Андрей Шеин
Component: WW_make Version:
Keywords: Cc:

Description


Change History (5)

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

Owner: changed from Дмитрий Лапшин (lapshin) to Андрей Шеин
Summary: WW #1WW #1 shein.andrey
Type: ожидается проверкаожидаются исправления

Привет! Makefile не очень правильный, он пересобирает каждый раз зачем-то всё пересобирает. Дело в том, что у тебя папку bin создаёт сам make, это мило, но ты зависишь от папки. А папка меняется каждый раз, когда в ней что-то меняют. Чтобы не было такой ерунды, в Makefile можно написать, что эта зависимость не должна быть новее, а просто должна быть:

a.o: a.c b.h … | bin
	gcc …

Ну или можно было в svn положить пустую папку, он это умеет. git не умеет.

Отступы слегка съехали в util.c.

У тебя забавные заголовочные файлы. Зачем main.h объявляет все эти фукнции? Заголовочные файлы описывают не что единице трансляции нужно, а что она может предложить другим. Тогда объявление всего одно, и все его используют.

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

Version: 1.0

У тебя Makefile неправильный. Цель должна называться как результат, путь целиком. А то как он поймёт, что ты собирал? Никак:

bin/a.o: …
	gcc … -o bin/a.o

Отступы в max неконсистенты. В заголовочных файлах аргументы функций бы прописать с именами.

comment:3 Changed 5 years ago by Андрей Шеин

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

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

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

В заголовочных файлах аргументы функций бы прописать с именами.

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

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

Дедлайн прошёл.

Note: See TracTickets for help on using tickets.