i thought it was 1 bit, not 1 byte... thanks for correcting my mistake :) On Tue, Nov 3, 2015 at 10:00 AM, Thomas Faber <thomas.fa...@reactos.org> wrote:
> It makes a difference because m_RefreshPending was initialized to 0 by > the constructor, but the three bytes behind it were not, so the first > cmpxchg could fail. > Plus of course if someone decided to add another bool or a char array > or whatever at the end, that would get overwritten. > > > On 2015-11-03 00:21, Timo Kreuzer wrote: > > In fact in this case it wouldn't make a difference, since the bool would > > be converted to a LONG. But using TRUE/FALSE seems to be appropriate > here. > > > > Am 22.10.2015 um 19:37 schrieb gedmur...@svn.reactos.org: > >> Author: gedmurphy > >> Date: Thu Oct 22 17:37:51 2015 > >> New Revision: 69650 > >> > >> URL: http://svn.reactos.org/svn/reactos?rev=69650&view=rev > >> Log: > >> The c++ bool is 1 byte, not 4. Thanks Thomas > >> > >> Modified: > >> trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.cpp > >> trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.h > >> > >> Modified: trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.cpp > >> URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.cpp?rev=69650&r1=69649&r2=69650&view=diff > >> > ============================================================================== > >> --- trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.cpp > [iso-8859-1] (original) > >> +++ trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.cpp > [iso-8859-1] Thu Oct 22 17:37:51 2015 > >> @@ -786,7 +786,7 @@ > >> // we set a timer to run in 500ms, which should leave > enough time for all > >> // the messages to come through. Wrap so we don't set > multiple timers > >> // > >> - if (InterlockedCompareExchange((LONG > *)&This->m_RefreshPending, true, false) == false) > >> + if (InterlockedCompareExchange((LONG > *)&This->m_RefreshPending, TRUE, FALSE) == FALSE) > >> { > >> SetTimer(hwnd, REFRESH_TIMER, 500, NULL); > >> } > >> @@ -807,7 +807,7 @@ > >> KillTimer(hwnd, REFRESH_TIMER); > >> > >> // Allow more change notifications > >> - InterlockedExchange((LONG *)&This->m_RefreshPending, > false); > >> + InterlockedExchange((LONG *)&This->m_RefreshPending, > FALSE); > >> } > >> break; > >> } > >> > >> Modified: trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.h > >> URL: > http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.h?rev=69650&r1=69649&r2=69650&view=diff > >> > ============================================================================== > >> --- trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.h > [iso-8859-1] (original) > >> +++ trunk/reactos/dll/win32/devmgr/devmgmt/MainWindow.h > [iso-8859-1] Thu Oct 22 17:37:51 2015 > >> @@ -17,7 +17,7 @@ > >> HMENU m_hMenu; > >> HMENU m_hActionMenu; > >> int m_CmdShow; > >> - bool m_RefreshPending; > >> + BOOL m_RefreshPending; > >> > >> public: > >> CDeviceManager(void); > >> > >> > >> > > _______________________________________________ > Ros-dev mailing list > Ros-dev@reactos.org > http://www.reactos.org/mailman/listinfo/ros-dev >
_______________________________________________ Ros-dev mailing list Ros-dev@reactos.org http://www.reactos.org/mailman/listinfo/ros-dev