diff --git a/dlls/comctl32/comctl32undoc.c b/dlls/comctl32/comctl32undoc.c
index e9a8453e3764153a3b899fb45e0de0d36c942f3f..b93d3d1bb783e220de70cc540588231e3cb78f23 100644
--- a/dlls/comctl32/comctl32undoc.c
+++ b/dlls/comctl32/comctl32undoc.c
@@ -51,7 +51,7 @@ typedef struct _LOADDATA
     PVOID ptr;
 } LOADDATA, *LPLOADDATA;
 
-typedef HRESULT CALLBACK (*DPALOADPROC)(LPLOADDATA,IStream*,LPARAM);
+typedef HRESULT (CALLBACK *DPALOADPROC)(LPLOADDATA,IStream*,LPARAM);
 
 INT __cdecl _wtoi(LPWSTR string);
 
@@ -264,7 +264,7 @@ DPA_Merge (const HDPA hdpa1, const HDPA hdpa2, DWORD dwFlags,
 	}
 	else if (nResult < 0)
 	{
-	    if (!dwFlags & 8)
+	    if (!(dwFlags & 8))
 	    {
 		PVOID ptr;
 
@@ -277,7 +277,7 @@ DPA_Merge (const HDPA hdpa1, const HDPA hdpa2, DWORD dwFlags,
 	}
 	else
 	{
-	    if (!dwFlags & 4)
+	    if (!(dwFlags & 4))
 	    {
 		PVOID ptr;
 
diff --git a/dlls/comctl32/header.c b/dlls/comctl32/header.c
index 744154364b2c3d9ba9cb45219d5f2198122ee374..d4cdda2a908e61b24c9c2f7807119d34be8fdc4c 100644
--- a/dlls/comctl32/header.c
+++ b/dlls/comctl32/header.c
@@ -883,7 +883,7 @@ HEADER_InsertItemA (HWND hwnd, WPARAM wParam, LPARAM lParam)
     if (lpItem->fmt == 0)
 	lpItem->fmt = HDF_LEFT;
 
-    if (!(lpItem->fmt &HDF_STRING) && (phdi->mask & HDI_TEXT))
+    if (!(lpItem->fmt & HDF_STRING) && (phdi->mask & HDI_TEXT))
       {
 	lpItem->fmt |= HDF_STRING;
       }
diff --git a/dlls/comctl32/rebar.c b/dlls/comctl32/rebar.c
index 309f3eae53c0697b0fe943bdb0294b23f41a97d6..63749b6763edaed083c4ad2fd1d4cc9408a7e9d3 100644
--- a/dlls/comctl32/rebar.c
+++ b/dlls/comctl32/rebar.c
@@ -1727,7 +1727,7 @@ REBAR_Layout (REBAR_INFO *infoPtr, LPRECT lpRect, BOOL notify, BOOL resetclient)
 	    iband = infoPtr->rows[i-1].istartband;
 	    lpBand = &infoPtr->bands[iband];
 	    if(HIDDENBAND(lpBand)) continue;
-	    if (!lpBand->fMask & RBBS_VARIABLEHEIGHT) continue;
+	    if (!(lpBand->fMask & RBBS_VARIABLEHEIGHT)) continue;
 	    if (((INT)lpBand->cyMaxChild < 1) || 
 		((INT)lpBand->cyIntegral < 1)) {
 		if (lpBand->cyMaxChild + lpBand->cyIntegral == 0) continue;
diff --git a/dlls/commdlg/fontdlg.c b/dlls/commdlg/fontdlg.c
index 28107cbdcd8380844e5fa8f8261fdb72814a3e9c..13579e9302223cba1e9a799e6a70223bf8b4d25a 100644
--- a/dlls/commdlg/fontdlg.c
+++ b/dlls/commdlg/fontdlg.c
@@ -636,9 +636,9 @@ static INT AddFontStyle(const LOGFONTA *lplf, UINT nFontType,
 
   if (!SendMessageA(hcmb2, CB_GETCOUNT, 0, 0))
   {
-       HDC hdc= (lpcf->Flags & CF_PRINTERFONTS && lpcf->hDC) ? lpcf->hDC : GetDC(hDlg);
+       HDC hdc= ((lpcf->Flags & CF_PRINTERFONTS) && lpcf->hDC) ? lpcf->hDC : GetDC(hDlg);
        i=SetFontStylesToCombo2(hcmb2,hdc,lplf);
-       if (!(lpcf->Flags & CF_PRINTERFONTS && lpcf->hDC))
+       if (!((lpcf->Flags & CF_PRINTERFONTS) && lpcf->hDC))
          ReleaseDC(hDlg,hdc);
        if (i)
         return 0;
@@ -733,7 +733,7 @@ static LRESULT CFn_WMInitDialog(HWND hDlg, WPARAM wParam, LPARAM lParam,
     ShowWindow(GetDlgItem(hDlg,grp1),SW_HIDE);
     ShowWindow(GetDlgItem(hDlg,stc4),SW_HIDE);
   }
-  hdc= (lpcf->Flags & CF_PRINTERFONTS && lpcf->hDC) ? lpcf->hDC : GetDC(hDlg);
+  hdc= ((lpcf->Flags & CF_PRINTERFONTS) && lpcf->hDC) ? lpcf->hDC : GetDC(hDlg);
   if (hdc)
   {
     CFn_ENUMSTRUCT s;
@@ -811,7 +811,7 @@ static LRESULT CFn_WMInitDialog(HWND hDlg, WPARAM wParam, LPARAM lParam,
     return FALSE;
   }
 
-  if (!(lpcf->Flags & CF_PRINTERFONTS && lpcf->hDC))
+  if (!((lpcf->Flags & CF_PRINTERFONTS) && lpcf->hDC))
     ReleaseDC(hDlg,hdc);
   SetCursor(hcursor);   
   return TRUE;
@@ -968,7 +968,7 @@ static LRESULT CFn_WMCommand(HWND hDlg, WPARAM wParam, LPARAM lParam,
   {
 	case cmb1:if (HIWORD(wParam)==CBN_SELCHANGE)
 		  {
-		    hdc=(lpcf->Flags & CF_PRINTERFONTS && lpcf->hDC) ? lpcf->hDC : GetDC(hDlg);
+		    hdc=((lpcf->Flags & CF_PRINTERFONTS) && lpcf->hDC) ? lpcf->hDC : GetDC(hDlg);
 		    if (hdc)
 		    {
                       SendDlgItemMessageA(hDlg, cmb2, CB_RESETCONTENT16, 0, 0); 
@@ -990,7 +990,7 @@ static LRESULT CFn_WMCommand(HWND hDlg, WPARAM wParam, LPARAM lParam,
 			SendDlgItemMessageA(hDlg,cmb3, CB_SETCURSEL, 0, 0);
 		        SetCursor(hcursor);
 		      }
-		      if (!(lpcf->Flags & CF_PRINTERFONTS && lpcf->hDC))
+		      if (!((lpcf->Flags & CF_PRINTERFONTS) && lpcf->hDC))
  		        ReleaseDC(hDlg,hdc);
  		    }
  		    else
diff --git a/dlls/shell32/iconcache.c b/dlls/shell32/iconcache.c
index cf3df2f489cc5d48e183fd336a8080105e486cd5..ac70554f74710334770436a5e2285c4628541740 100644
--- a/dlls/shell32/iconcache.c
+++ b/dlls/shell32/iconcache.c
@@ -405,7 +405,7 @@ HICON WINAPI ExtractIconExA ( LPCSTR lpszFile, INT nIconIndex, HICON * phiconLar
 	}
 
 	/* if no pointers given and one icon expected, return the handle directly*/
-	if (!phiconLarge && ! phiconSmall && nIcons==1 )
+	if (!phiconLarge && !phiconSmall && nIcons==1 )
 	  phiconSmall = &ret;
 	
 	if (phiconSmall)
diff --git a/dlls/user/dde/client.c b/dlls/user/dde/client.c
index 2bf99a55d1ac3d528ef931172ab0ab746e2664f1..ee0a63d5d7bdd41b7d646a91b99eacb82f6aace7 100644
--- a/dlls/user/dde/client.c
+++ b/dlls/user/dde/client.c
@@ -1225,7 +1225,7 @@ BOOL WINAPI DdeAbandonTransaction(DWORD idInst, HCONV hConv, DWORD idTransaction
         {
             for (pConv = pInstance->convs[WDML_CLIENT_SIDE]; pConv; pConv = pConv->next)
             {
-                if (!pConv->wStatus & ST_CONNECTED) continue;
+                if (!(pConv->wStatus & ST_CONNECTED)) continue;
                 for (pXAct = pConv->transactions; pXAct; pXAct = pXAct->next)
                 {
                     if (pXAct->dwTimeout == TIMEOUT_ASYNC)
diff --git a/tools/winedump/msmangle.c b/tools/winedump/msmangle.c
index adee2ac69ab765f434cab8236ef330820fa5280e..109f9a48d367804c5f5b4b83bc2939b736fa137a 100644
--- a/tools/winedump/msmangle.c
+++ b/tools/winedump/msmangle.c
@@ -453,7 +453,7 @@ static char *demangle_datatype (char **str, compound_type *ct,
           fatal ("Out of Memory");
 
         /* If we're a reference, re-use the pointer already in the type */
-        if (!ct->flags & CT_BY_REFERENCE)
+        if (!(ct->flags & CT_BY_REFERENCE))
           stripped[ strlen (stripped) - 2] = '\0'; /* otherwise, strip it */
 
         ct->expression = str_create (2, ct->flags & CT_CONST ? "const " :