Am 03.11.2015 um 22:09 schrieb Thomas Faber:
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? ;)
No, I just wanted to test you. *cough* ;-)


Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

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

Reply via email to