Я пытаюсь собрать openrave на ветке production
и не могу понять, что вижу. Код выглядит так, как будто он не включает #include <mutex>
, но std::mutex
используется повсюду. Я уверен, что что-то упускаю. Как это возможно? Как код может использовать std::mutex
, не включая его напрямую?
Одним из примеров является файл Sensorsystem.h (include/openrave/sensorsystem.h) по адресу https://github.com/rdiankov/openrave/tree/production/include/openrave.
Любая помощь будет оценена по достоинству.
Включение любого заголовка стандартной библиотеки (как правило) может привести или не привести к включению объявлений любого другого заголовка стандартной библиотеки. Он не указан и будет различаться в зависимости от реализации. Таким образом, кому-то может повезти (или не повезти?), если вы забудете включить заголовок, который должен был быть включен, и он все еще работает. Это может работать на одном компиляторе, но не на другом. Это следует исправить, если действительно отсутствует включение <mutex>
.
Возможно, заголовок thread
включает его неявно. В любом случае код плохой и не всегда гарантированно компилируется.
Во всех их файлах есть #include <openrave/openrave.h> , который имеет #include <openrave/environment.h>, этот файл содержит
#include <mutex>
Я думаю, что это, наверное, правильный ответ. Это кажется крайне схематичным, поскольку условие препроцессора в этом файле не всегда включает <мьютекс> перед его использованием. Кроме того, я подумал, что вам всегда следует указывать то, что вы используете, прежде чем использовать это. Более того, я не знал, что можно использовать что-то вроде объявления std::mutex в файле заголовка, если соответствующий файл cpp содержит соответствующий заголовок для мьютекса. Кто-нибудь, пожалуйста, поправьте меня, но это кажется плохой практикой и может привести к проблемам с компиляцией.
Включение заголовка @Pinknoise — это просто замена текста файла заголовка в файл .cpp
. Важно то, появляется ли объявление перед первым использованием после замены текста (рекурсивно). Если перед первым объявлением будет использование, программа будет неправильно сформирована. В языке C++ фактически нет понятия разницы между файлами .h
и .cpp
.
@Pinknoise Поскольку вы никогда не знаете, какие заголовки библиотеки включают в себя какие заголовки других библиотек в качестве детали реализации, вы никогда не можете рассчитывать на то, что код не компилируется, если вы забудете включить соответствующие заголовки или включите их недостаточно рано. Для стандартной библиотеки это явно не указано (по большей части), а для сторонних библиотек также редко есть какие-либо гарантии того, что один заголовок не включает другой. Поэтому каждый раз, когда вы используете новую функциональность библиотеки, следует проверять, требуется ли формально дополнительное включение, даже если она компилируется без него.
Спасибо всем за ваши комментарии. Теперь все это имеет смысл.
Вы также можете поднять проблему в репозитории openrave и спросить их.