On 24.11.2011 3:27, Jérôme Gardou wrote:
-            *ShortPtr = SWAPW(*ShortPtr) + LOWORD(Delta);
+ *ShortPtr = SWAPW(*ShortPtr) + LOWORD(Delta& 0xFFFF);
Maybe turn off this "feature" of MSVC checker then? Excuse me, but it's literally a code pollution. Especially the above case, of firstly ANDing with 0xFFFF, and *then* applying LOWORD (which would not change anything at all).

Or, as Timo said, maybe fix the macro? Move the cast elsewhere, so at first you &, and only then cast?


WBR,
Aleskey.

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

Reply via email to