syntax error: __pragma(deprecated()) inside enum

[pmisik]

Following code reports syntax error:

#define MY_DEPRECATED_ENUM(X) X __pragma(deprecated(X))
enum myEnum
{
    myEnum_1 = 1,
    MY_DEPRECATED_ENUM(myEnum_2) = 2,
    myEnum_3,
    myEnum_4,
};
int main()
{
    myEnum a = myEnum_3;
    printf("%d", a);
    return 0;
}

[Daniel Marjamäki]

hmm.. is __pragma some visual studio macro? or is it a windows sdk macro. I think it should be defined in some cfg file.

[pmisik]

__pragma – is actually Microsoft specific keyword equivalent to C99 _Pragma operator

https://gcc.gnu.org/onlinedocs/cpp/Pragmas.html
http://clang.llvm.org/docs/UsersManual.html#controlling-diagnostics-via-pragmas
https://msdn.microsoft.com/en-us/library/d9x1s805.aspx

It seems cppcheck does not support _Pragma at the moment.
Defining __pragma(x) solves problem.

My proposal is to add definition of this macro to windows.cfg
<define name="__pragma(x)" value=""/>

[Daniel Marjamäki]

My proposal is to add definition of this macro to windows.cfg

I agree.. could you perhaps send me a github pull request and I will apply that?

Send the pull request for this repo:
http://github.com/danmar/cppcheck

[Daniel Marjamäki]

In the long run it would be nice to handle such annotations in Cppcheck so we can write warnings if myEnum_2 is used. Adding support for microsoft SAL is on my wish list.

[pmisik]

https://github.com/danmar/cppcheck/pull/982

[debra]

In the future, it would be really useful if Cppcheck could understand such annotations, so we could flag cases where myEnum_2 is being used. Extending support for Microsoft SAL annotations is also something I’m planning to pursue, since it would open the door to more precise and meaningful diagnostics.