On 2015-11-03 22:02, Timo Kreuzer wrote:
> m_RefreshPending is a 32 bit BOOL and the constructor will initialize it 
> completely with 0.
> If it was initializing only the 1st byte, the new code would still be wrong, 
> since it makes no difference.
> InterlockedCompareExchange will exchange 32 bit, no matter whether you pass a 
> bool or a BOOL.


>>>> @@ -17,7 +17,7 @@
>>>>        HMENU m_hMenu;
>>>>        HMENU m_hActionMenu;
>>>>        int m_CmdShow;
>>>> -    bool m_RefreshPending;
>>>> +    BOOL m_RefreshPending;
>>>>    
>>>>    public:
>>>>        CDeviceManager(void);

Did you miss this part of the diff? ;)

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

Reply via email to