Мы пытаемся выполнить статический анализ, используя cppcheck для нашей базы кода для Linux, используя плагин jenkins. По какой-то причине он находит множество ошибок для следующего типа вещей:
CREATE_DERIVED_EXCEPTION_CLASS(ExceptionOSApiError, 5)
а также для:
CREATE_EXCEPTION_CLASS_DERIVED_FROM_SPECIFIC_BASE(ExceptionFileApiError, ExceptionOSApiError, 6)
Где он определяется как (но не дает ошибки в строках):
#define CREATE_DERIVED_EXCEPTION_CLASS( new_exception_name, unique_numeric_code ) \
CREATE_EXCEPTION_CLASS_DERIVED_FROM_SPECIFIC_BASE( new_exception_name, Exception, unique_numeric_code )
#ifdef _IN_EXCEPTION_CPP
#define CREATE_EXCEPTION_CLASS_DERIVED_FROM_SPECIFIC_BASE( new_exception_name, base_exception_name, unique_numeric_code ) \
new_exception_name::new_exception_name( const char *message, LASTERROR_TYPE lastError, const IDebugContext& debugContextWhereThrown ) \
: base_exception_name( message, lastError, debugContextWhereThrown ) {} \
new_exception_name::new_exception_name( std::string&& message, LASTERROR_TYPE lastError, const IDebugContext& debugContextWhereThrown ) \
: base_exception_name( std::move(message), lastError, debugContextWhereThrown ) {} \
new_exception_name::new_exception_name( LASTERROR_TYPE lastError, const IDebugContext& debugContextWhereThrown ) \
: base_exception_name( lastError, debugContextWhereThrown ) {} \
new_exception_name::new_exception_name(new_exception_name&& source) \
: base_exception_name(std::move(source)) {} \
new_exception_name& new_exception_name::operator=(new_exception_name&& rightHandSide) \
{ \
if (this != &rightHandSide) \
{ \
base_exception_name::operator=(std::move(rightHandSide)); \
/* No derived class data members to move */ \
} \
return(*this); \
} \
UTILDLL_EXPORT int new_exception_name::getExceptionTypeByNumericCode() const \
{ \
return( unique_numeric_code ); /* This must be UNIQUE! */ \
}
#else // !_IN_CPEXCEPTION_CPP
#define CREATE_EXCEPTION_CLASS_DERIVED_FROM_SPECIFIC_BASE( new_exception_name, base_exception_name, unique_numeric_code ) \
class UTILDLL_EXPORT new_exception_name : public base_exception_name \
{ \
public: \
new_exception_name( const char *message, LASTERROR_TYPE lastError, const IDebugContext& debugContextWhereThrown ); \
new_exception_name( std::string&& message, LASTERROR_TYPE lastError, const IDebugContext& debugContextWhereThrown ); \
new_exception_name( LASTERROR_TYPE lastError, const IDebugContext& debugContextWhereThrown ); \
new_exception_name( const new_exception_name& source ) = default; \
new_exception_name& operator=(const new_exception_name& rightHandSide) = default; \
new_exception_name(new_exception_name&& source); \
new_exception_name& operator=(new_exception_name&& rightHandSide); \
virtual ~new_exception_name() = default; \
virtual int getExceptionTypeByNumericCode() const; \
};
#endif // !_IN_EXCEPTION_CPP
Есть идеи?
Я нашел этот тип информации в Интернете: избыточен с прагмой один раз ==(Похоже, есть известная проблема с cppcheck и #pragma Once (#ifndef #define - это исправление, но я не думаю, что мы этого хотим). Я думаю, что мы' я бы увидел намного больше этой проблемы, если бы это было #pragma когда-то.
Вот вторая заметка о проблеме с избыточным кодом: enum или ifndef о проблеме с избыточным кодом = =(Я не думаю, что это проблема с перечислением) (может быть, у нас нет #ifndef #define the .h … я его не вижу)
cppcheck считает, что у меня есть «Избыточный код: найден оператор который начинается с числовой константы»
Весь файл .h, в котором они определены, содержит только макросы и их использование. Владелец не думает, что ему нужен #ifndef вокруг #define, так как cppcheck жалуется на все строки под #define, которые используют макросы.
Кроме того: вокруг всего файла .h нет #ifndef, но владелец не считает это необходимым.
Информация о сборке: Обычно для нашей сборки:
g++ -std=c++11 -ggdb -Wall -Wextra -Werror -pedantic -fdiagnostic-show-option -fPIC -DLinuxx86_64
Что я вижу в jenkins для настройки cppcheck:
cppcheck --enable=all --xml --xml-version=2 -DLinuxx86_64 --platform=unix64 --include=Src/Headers/CommonPrecomp.h -v --report-progress -DOPTIMUS_CDECL -DUTILDLL_EXPORT -DOPTIMUS_COREDLL_EXPORT -DREADERDLL_EXPORT -DWRITERDLL_EXPORT -E Src 2> cppcheck.xml
Я не уверен, что сборка jenkins для cppcheck выполняется так же, как и для нашей обычной сборки.
Любые другие идеи? Спасибо!
CREATE_EXCEPTION_CLASS_DERIVED_FROM_SPECIFIC_BASE
для отработки сгенерированного кода. Также попробуйте сохранить предварительно обработанный файл (если расширения макросов слишком сложны). - person Richard Critten   schedule 17.09.2015unique_numeric_code
. Это не проблема, просто наблюдение. - person Richard Critten   schedule 17.09.2015-E
gcc, чтобы все макросы препроцессора были расширены (другой компилятор может иметь аналогичные переключатели), и запустите cppcheck для полученного файла. - person Petr   schedule 17.09.2015