cppcheck обнаружил избыточный код: нашел оператор, начинающийся с числовой константы для макроса

Мы пытаемся выполнить статический анализ, используя 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 выполняется так же, как и для нашей обычной сборки.

Любые другие идеи? Спасибо!


person Michele    schedule 17.09.2015    source источник
comment
Нам также нужно определение CREATE_EXCEPTION_CLASS_DERIVED_FROM_SPECIFIC_BASE для отработки сгенерированного кода. Также попробуйте сохранить предварительно обработанный файл (если расширения макросов слишком сложны).   -  person Richard Critten    schedule 17.09.2015
comment
Не видно, где используется unique_numeric_code. Это не проблема, просто наблюдение.   -  person Richard Critten    schedule 17.09.2015
comment
Я добавил CREATE_EXCEPTION_CLASS_DERIVED_FROM_SPECIFIC_BASE выше в исходный вопрос.   -  person Michele    schedule 17.09.2015
comment
Я только что нашел дополнительную информацию о #else и добавил ее в раздел кода.   -  person Michele    schedule 17.09.2015
comment
Попробуйте скомпилировать свой код с переключателем -E gcc, чтобы все макросы препроцессора были расширены (другой компилятор может иметь аналогичные переключатели), и запустите cppcheck для полученного файла.   -  person Petr    schedule 17.09.2015
comment
Я добавил информацию о сборке выше. Я не вижу, чтобы компилятор gcc использовался, когда мы запускаем cppcheck. Как вы думаете, нужно ли что-то изменить, когда мы запускаем cppcheck?   -  person Michele    schedule 17.09.2015


Ответы (1)


Похоже, проблема заключалась в том, что он не попадал в наши включаемые каталоги. Мы их принесли из файла. Нам пришлось добавить их в настройку сборки jenkins:

cppcheck -j 2 --xml --xml-version=1 --enable=all -DLinuxx86_64 --platform=unix64 -v --report-progress --suppress=missingIncludeSystem --include=Sr/Head/Common.h -i Sr/Doc/Win.cpp -i Sr/P/S/S.cpp -DOPT_CDECL -DUTILDLL_EXPORT -DOPT_COREDLL_EXPORT -DREADLL_EXPORT -DWDLL_EXPORT -ISr/Head -ISr/Head/libopc -ISr/Head/lib/config -ISrc/Head/lib/lib2 -ISr/Snot -ISr/Doc -ISr/Doc -ISr/PDers/XP -ISr/PDers/Ca -ISr/PDers/Ca/Head -IVersionAndBuildConfig -ISr/Head/GoogleTest/include -ISr/Head/GoogleTest/include/gtest Sr/Doc > cppcheck.xml
person Michele    schedule 17.09.2015