cheng/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix undefined behaviour due to using shift variable twice in the same expression

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@41443 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin 2006-09-25 18:15:39 +00:00
parent 7b514b3b43
commit 325125ba4f
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/cocoa/cursor.mm
View File

@ -212,8 +212,9 @@ NSCursor* wxGetStockCursor( short sIndex )
//do the rest of those bits and alphas :) //do the rest of those bits and alphas :)
for (int shift = 0; shift < 32; ++shift) for (int shift = 0; shift < 32; ++shift)
{ {
data[i] |= ( !!( (pCursor->mask[i] & (1 << (shift >> 1) )) ) ) << shift; const int bit = 1 << (shift >> 1);
data[i] |= ( !( (pCursor->bits[i] & (1 << (shift >> 1) )) ) ) << ++shift; data[i] |= ( !!( (pCursor->mask[i] & bit) ) ) << shift;
data[i] |= ( !( (pCursor->bits[i] & bit) ) ) << ++shift;
} }
} }