Lagman 1 2 декабря, 2016 Опубликовано 2 декабря, 2016 · Жалоба Посмотрел код, из проекта Open Pilot от Geohot, https://github.com/commaai/openpilot/tree/master/board и как то не понял, код оформлен (ну пускай авто отформатирован), но зачем он так сделал, в заголовочных .h файлах лежат куски кода и потом все эти файлы включаются в main.c Да, это будет работать и даже откомпилируется (наверно с множеством предупреждений т.к. прототипов функций нет, хотя может он их по порядку выставил :)), но это все равно что весь код разместить в одном файле и дальнейшая модификация будет затруднена. (Это все по моему мнению.) Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Eddy_Em 1 2 декабря, 2016 Опубликовано 2 декабря, 2016 · Жалоба Дикий изврат. Мне только один раз понадобилось код размещать в инклюдируемых файлах — для реализации операций поиска связных компонет. Код почти одинаковый, различающийся совсем немногим для 4- и 8-связных областей, поэтому решил, что легче будет с разными дефайнами заинклюдить один и тот же файл дважды! // на гитхабе у меня лежит. Кажись, в с-сниппетах. Не помню уж точно. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
becopt 0 8 декабря, 2016 Опубликовано 8 декабря, 2016 · Жалоба ИМХО, куски кода в хедерах нужны только в случае, если код нужно инлайнить, т.е. описать как static inline и проблем быть не должно. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
smalcom 0 5 февраля, 2017 Опубликовано 5 февраля, 2017 · Жалоба но зачем он так сделал, в заголовочных .h файлах лежат куски кода и потом все эти файлы включаются в main.c Плохо знает используемый компилятор. Как только пойдут множественные включения в несколько компилируемых единиц, то появятся ошибки линковки. Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
Kabdim 0 5 февраля, 2017 Опубликовано 5 февраля, 2017 · Жалоба Может попытка добиться массового инлайна до того как в гцц толково заработла опция lto (а она кстати совсем недавно заработала без граблей). Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться
psL 0 14 февраля, 2017 Опубликовано 14 февраля, 2017 · Жалоба автору было лень с Makefile заморачиваться Цитата Поделиться сообщением Ссылка на сообщение Поделиться на другие сайты Поделиться