Opened 5 years ago

Closed 5 years ago

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

WW #1 anisimova.karina

Reported by: anisimova.karina Owned by: Дмитрий Лапшин (lapshin)
Component: WW_make Version:
Keywords: Cc:

Description


Change History (7)

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

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

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

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

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

Рекомендую чуть аккуратнее следовать стилю: пробелы правильнее расставлять, пустые строчки где нужно бы что-то отделить.

comment:2 Changed 5 years ago by anisimova.karina

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

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

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

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

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

comment:4 Changed 5 years ago by anisimova.karina

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

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

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

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

comment:6 Changed 5 years ago by anisimova.karina

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

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

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

Обычно мы свои инклуды включаем в кавычках, а не в треугольничках, но пойдёт.

Note: See TracTickets for help on using tickets.