On 2011-11-17 12:07, Timo Kreuzer wrote:
> Am 16.11.2011 22:17, schrieb tfa...@svn.reactos.org:
>> -#define SharedUserData                  ((KUSER_SHARED_DATA 
>> *CONST)USER_SHARED_DATA)
>> +#define SharedUserData                  ((KUSER_SHARED_DATA 
>> *)USER_SHARED_DATA)
>
> It is like this (with the const) in wdm.h
>
> #define SharedUserData  ((KUSER_SHARED_DATA * const) KI_USER_SHARED_DATA)
>
> maybe its supposed to express that "SharedUserData" is a constant and not
> a variable... although that doesn't make much sense

Hmm. Interesting.
Didn't spot it in wdm.h. Thanks!

Apparently, MS doesn't use Coverity (what a shock :D)

It could certainly be argued that it has some value as an annotation.
But since the const is neither wrong in any way nor makes any difference at
all, I think ~300 Coverity parse warnings are a good reason to remove it.

I'll fix it in our wdm.h too later, unless someone disagrees.

Thanks,

Tom

_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to