'sizeof type' is best avoided as it won't always compile (e.g. 'int
main() {return sizeof int;}'). Use 'sizeof(type)' instead.
diff --git a/dlls/advapi32/registry.c b/dlls/advapi32/registry.c
index ff890a3..e4ea1c9 100644
--- a/dlls/advapi32/registry.c
+++ b/dlls/advapi32/registry.c
@@ -797,7 +797,7 @@
{
WCHAR name[MAX_PATH];
- while(!RegEnumKeyW(tmp, 0, name, sizeof name))
+ while(!RegEnumKeyW(tmp, 0, name, sizeof(name)))
{
if(RegDeleteKeyW(tmp, name)) /* recurse */
break;
@@ -831,7 +831,7 @@
{
CHAR name[MAX_PATH];
- while(!RegEnumKeyA(tmp, 0, name, sizeof name))
+ while(!RegEnumKeyA(tmp, 0, name, sizeof(name)))
{
if(RegDeleteKeyA(tmp, name)) /* recurse */
break;
diff --git a/dlls/advapi32/service.c b/dlls/advapi32/service.c
index 26b5a62..bcc4542 100644
--- a/dlls/advapi32/service.c
+++ b/dlls/advapi32/service.c
@@ -624,11 +624,11 @@
TRACE("(%p,%ld,%p)\n",hService,dwNumServiceArgs,
lpServiceArgVectors);
- size = sizeof str;
+ size = sizeof(str);
r = RegQueryValueExW(hService, _ImagePathW, NULL, &type, (LPVOID)str, &size);
if (r!=ERROR_SUCCESS)
return FALSE;
- ExpandEnvironmentStringsW(str,path,sizeof path);
+ ExpandEnvironmentStringsW(str,path,sizeof(path));
TRACE("Starting service %s\n", debugstr_w(path) );
@@ -718,7 +718,7 @@
FIXME("(%p,%p) partial\n",hService,lpservicestatus);
/* read the service type from the registry */
- size = sizeof val;
+ size = sizeof(val);
r = RegQueryValueExA(hService, "Type", NULL, &type, (LPBYTE)&val, &size);
if(type!=REG_DWORD)
{
diff --git a/dlls/comcat/information.c b/dlls/comcat/information.c
index f2caf5b..9841525 100644
--- a/dlls/comcat/information.c
+++ b/dlls/comcat/information.c
@@ -955,7 +955,7 @@
WCHAR prefix[6] = { 'C', 'L', 'S', 'I', 'D', '\\' };
This->lpVtbl = &COMCAT_CATID_IEnumGUID_Vtbl;
- memcpy(This->keyname, prefix, sizeof prefix);
+ memcpy(This->keyname, prefix, sizeof(prefix));
StringFromGUID2(rclsid, This->keyname + 6, 39);
lstrcpyW(This->keyname + 44, postfix);
RegOpenKeyExW(HKEY_CLASSES_ROOT, This->keyname, 0, KEY_READ, &This->key);
diff --git a/dlls/comcat/regsvr.c b/dlls/comcat/regsvr.c
index 2406fc0..9256796 100644
--- a/dlls/comcat/regsvr.c
+++ b/dlls/comcat/regsvr.c
@@ -370,7 +370,7 @@
HKEY subkey;
for (;;) {
- cName = sizeof subkey_name / sizeof(WCHAR);
+ cName = sizeof(subkey_name) / sizeof(WCHAR);
res = RegEnumKeyExW(key, 0, subkey_name, &cName,
NULL, NULL, NULL, NULL);
if (res != ERROR_SUCCESS && res != ERROR_MORE_DATA) {
diff --git a/dlls/comctl32/comctl32undoc.c b/dlls/comctl32/comctl32undoc.c
index 73ef986..7fda24c 100644
--- a/dlls/comctl32/comctl32undoc.c
+++ b/dlls/comctl32/comctl32undoc.c
@@ -2081,7 +2081,7 @@
if(pfnCompare(lpPtrs[l],lpPtrs[m+1],lParam)>0)
{
t = lpPtrs[m+1];
- memmove(&lpPtrs[l+1],&lpPtrs[l],(m-l+1)*sizeof lpPtrs[l]);
+ memmove(&lpPtrs[l+1],&lpPtrs[l],(m-l+1)*sizeof(lpPtrs[l]));
lpPtrs[l] = t;
m++;
diff --git a/dlls/comctl32/propsheet.c b/dlls/comctl32/propsheet.c
index 61da189..702e27b 100644
--- a/dlls/comctl32/propsheet.c
+++ b/dlls/comctl32/propsheet.c
@@ -503,7 +503,7 @@
if ( !HIWORD( lppsp->pszTitle ) )
{
- if (!LoadStringW( lppsp->hInstance, (UINT)lppsp->pszTitle,szTitle,sizeof szTitle ))
+ if (!LoadStringW( lppsp->hInstance, (UINT)lppsp->pszTitle,szTitle,sizeof(szTitle) ))
{
pTitle = pszNull;
FIXME("Could not load resource #%04x?\n",LOWORD(lppsp->pszTitle));
@@ -1995,7 +1995,7 @@
{
WCHAR szTitle[256];
MultiByteToWideChar(CP_ACP, 0, lpszText, -1,
- szTitle, sizeof szTitle);
+ szTitle, sizeof(szTitle));
PROPSHEET_SetTitleW(hwndDlg, dwStyle, szTitle);
}
else
diff --git a/dlls/comctl32/tooltips.c b/dlls/comctl32/tooltips.c
index 9384009..cb5ad96 100644
--- a/dlls/comctl32/tooltips.c
+++ b/dlls/comctl32/tooltips.c
@@ -2091,7 +2091,7 @@
TOOLTIPS_INFO *infoPtr = TOOLTIPS_GetInfoPtr (hwnd);
LOGFONTW lf;
- if(!GetObjectW((HFONT)wParam, sizeof lf, &lf))
+ if(!GetObjectW((HFONT)wParam, sizeof(lf), &lf))
return 0;
if(infoPtr->hFont) DeleteObject (infoPtr->hFont);
diff --git a/dlls/ddraw/ddraw/thunks.c b/dlls/ddraw/ddraw/thunks.c
index f3f3d2e..80e3161 100644
--- a/dlls/ddraw/ddraw/thunks.c
+++ b/dlls/ddraw/ddraw/thunks.c
@@ -329,8 +329,8 @@
DDSURFACEDESC DDSD;
struct displaymodescallback_context *cbcontext = context;
- memcpy(&DDSD,pDDSD2,sizeof DDSD);
- DDSD.dwSize = sizeof DDSD;
+ memcpy(&DDSD,pDDSD2,sizeof(DDSD));
+ DDSD.dwSize = sizeof(DDSD);
return cbcontext->func(&DDSD, cbcontext->context);
}
diff --git a/dlls/kernel/sync.c b/dlls/kernel/sync.c
index 22749e8..3a6ac7a 100644
--- a/dlls/kernel/sync.c
+++ b/dlls/kernel/sync.c
@@ -848,7 +848,7 @@
TRACE("%s 0x%08lx\n",debugstr_w(name),nTimeOut);
- memset(&ov,0,sizeof ov);
+ memset(&ov,0,sizeof(ov));
ov.hEvent = CreateEventA( NULL, 0, 0, NULL );
if (!ov.hEvent)
return FALSE;
@@ -917,7 +917,7 @@
return FALSE;
}
- memset(&ov,0,sizeof ov);
+ memset(&ov,0,sizeof(ov));
ov.hEvent = CreateEventA(NULL,0,0,NULL);
if (!ov.hEvent)
return FALSE;
diff --git a/dlls/kernel/tests/format_msg.c b/dlls/kernel/tests/format_msg.c
index adce921..f211fa4 100644
--- a/dlls/kernel/tests/format_msg.c
+++ b/dlls/kernel/tests/format_msg.c
@@ -43,176 +43,176 @@
/* the basics */
r = FormatMessageA(FORMAT_MESSAGE_FROM_STRING, "test", 0,
- 0, out, sizeof out/sizeof (CHAR),NULL);
+ 0, out, sizeof(out)/sizeof(CHAR),NULL);
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* using the format feature */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!s!", 0,
- 0, out, sizeof out/sizeof (CHAR), "test");
+ 0, out, sizeof(out)/sizeof(CHAR), "test");
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* no format */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1", 0,
- 0, out, sizeof out/sizeof (CHAR), "test");
+ 0, out, sizeof(out)/sizeof(CHAR), "test");
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* two pieces */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1%2", 0,
- 0, out, sizeof out/sizeof (CHAR), "te","st");
+ 0, out, sizeof(out)/sizeof(CHAR), "te","st");
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* three pieces */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1%3%2%1", 0,
- 0, out, sizeof out/sizeof (CHAR), "t","s","e");
+ 0, out, sizeof(out)/sizeof(CHAR), "t","s","e");
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* s doesn't seem to work in format strings */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%!s!", 0,
- 0, out, sizeof out/sizeof (CHAR), "test");
+ 0, out, sizeof(out)/sizeof(CHAR), "test");
ok(!strcmp("!s!", out),"failed out=[%s]",out);
ok(r==3,"failed: r=%ld",r);
/* S is unicode */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!S!", 0,
- 0, out, sizeof out/sizeof (CHAR), szwTest);
+ 0, out, sizeof(out)/sizeof(CHAR), szwTest);
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* as characters */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!c!%2!c!%3!c!%1!c!", 0,
- 0, out, sizeof out/sizeof (CHAR), 't','e','s');
+ 0, out, sizeof(out)/sizeof(CHAR), 't','e','s');
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* some numbers */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!d!%2!d!%3!d!", 0,
- 0, out, sizeof out/sizeof (CHAR), 1,2,3);
+ 0, out, sizeof(out)/sizeof(CHAR), 1,2,3);
ok(!strcmp("123", out),"failed out=[%s]",out);
ok(r==3,"failed: r=%ld",r);
/* a single digit with some spacing */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!4d!", 0,
- 0, out, sizeof out/sizeof (CHAR), 1);
+ 0, out, sizeof(out)/sizeof(CHAR), 1);
ok(!strcmp(" 1", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* a single digit, left justified */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!-4d!", 0,
- 0, out, sizeof out/sizeof (CHAR), 1);
+ 0, out, sizeof(out)/sizeof(CHAR), 1);
ok(!strcmp("1 ", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* two digit decimal number */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!4d!", 0,
- 0, out, sizeof out/sizeof (CHAR), 11);
+ 0, out, sizeof(out)/sizeof(CHAR), 11);
ok(!strcmp(" 11", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* a hex number */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!4x!", 0,
- 0, out, sizeof out/sizeof (CHAR), 11);
+ 0, out, sizeof(out)/sizeof(CHAR), 11);
ok(!strcmp(" b", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* a hex number, upper case */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!4X!", 0,
- 0, out, sizeof out/sizeof (CHAR), 11);
+ 0, out, sizeof(out)/sizeof(CHAR), 11);
ok(!strcmp(" B", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* a hex number, upper case, left justified */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!-4X!", 0,
- 0, out, sizeof out/sizeof (CHAR), 11);
+ 0, out, sizeof(out)/sizeof(CHAR), 11);
ok(!strcmp("B ", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* a long hex number, upper case */
r = doit(FORMAT_MESSAGE_FROM_STRING, "%1!4X!", 0,
- 0, out, sizeof out/sizeof (CHAR), 0x1ab);
+ 0, out, sizeof(out)/sizeof(CHAR), 0x1ab);
ok(!strcmp(" 1AB", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* two percent... */
r = doit(FORMAT_MESSAGE_FROM_STRING, " %%%% ", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp(" %% ", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* periods are special cases */
r = doit(FORMAT_MESSAGE_FROM_STRING, " %.%. %1!d!", 0,
- 0, out, sizeof out/sizeof (CHAR), 0x1ab);
+ 0, out, sizeof(out)/sizeof(CHAR), 0x1ab);
ok(!strcmp(" .. 427", out),"failed out=[%s]",out);
ok(r==7,"failed: r=%ld",r);
/* %0 ends the line */
r = doit(FORMAT_MESSAGE_FROM_STRING, "test%0test", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("test", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* %! prints an exclaimation */
r = doit(FORMAT_MESSAGE_FROM_STRING, "yah%!%0 ", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("yah!", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* %space */
r = doit(FORMAT_MESSAGE_FROM_STRING, "% % ", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp(" ", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING, "hi\n", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("hi\r\n", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* carriage return line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING, "hi\r\n", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("hi\r\n", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* carriage return line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING, "\r", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("\r\n", out),"failed out=[%s]",out);
ok(r==2,"failed: r=%ld",r);
/* carriage return line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING, "\r\r\n", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("\r\n\r\n", out),"failed out=[%s]",out);
ok(r==4,"failed: r=%ld",r);
/* change of pace... test the low byte of dwflags */
/* line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING | FORMAT_MESSAGE_MAX_WIDTH_MASK, "hi\n", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("hi ", out) || !strcmp("hi\r\n", out),"failed out=[%s]",out);
ok(r==3 || r==4,"failed: r=%ld",r);
/* carriage return line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING | FORMAT_MESSAGE_MAX_WIDTH_MASK, "hi\r\n", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp("hi ", out),"failed out=[%s]",out);
ok(r==3,"failed: r=%ld",r);
/* carriage return line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING | FORMAT_MESSAGE_MAX_WIDTH_MASK, "\r", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp(" ", out),"failed out=[%s]",out);
ok(r==1,"failed: r=%ld",r);
/* carriage return line feed */
r = doit(FORMAT_MESSAGE_FROM_STRING | FORMAT_MESSAGE_MAX_WIDTH_MASK, "\r\r\n", 0,
- 0, out, sizeof out/sizeof (CHAR));
+ 0, out, sizeof(out)/sizeof(CHAR));
ok(!strcmp(" ", out),"failed out=[%s]",out);
ok(r==2,"failed: r=%ld",r);
}
diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c
index 4f390df..c985413 100644
--- a/dlls/ntdll/file.c
+++ b/dlls/ntdll/file.c
@@ -217,8 +217,8 @@
FileHandle,EventHandle,ApcRoutine,ApcContext,IoStatusBlock,Buffer,Length,ByteOffset,Key);
if (IsBadWritePtr( Buffer, Length ) ||
- IsBadWritePtr( IoStatusBlock, sizeof *IoStatusBlock) ||
- IsBadWritePtr( ByteOffset, sizeof *ByteOffset) )
+ IsBadWritePtr( IoStatusBlock, sizeof(*IoStatusBlock)) ||
+ IsBadWritePtr( ByteOffset, sizeof(*ByteOffset)) )
return STATUS_ACCESS_VIOLATION;
IoStatusBlock->Information = 0;
diff --git a/dlls/ole32/regsvr.c b/dlls/ole32/regsvr.c
index a2fa984..9c91785 100644
--- a/dlls/ole32/regsvr.c
+++ b/dlls/ole32/regsvr.c
@@ -369,7 +369,7 @@
HKEY subkey;
for (;;) {
- cName = sizeof subkey_name / sizeof(WCHAR);
+ cName = sizeof(subkey_name) / sizeof(WCHAR);
res = RegEnumKeyExW(key, 0, subkey_name, &cName,
NULL, NULL, NULL, NULL);
if (res != ERROR_SUCCESS && res != ERROR_MORE_DATA) {
diff --git a/dlls/ole32/storage32.c b/dlls/ole32/storage32.c
index 32edea3..53c9422 100644
--- a/dlls/ole32/storage32.c
+++ b/dlls/ole32/storage32.c
@@ -6553,7 +6553,7 @@
if( string )
len = WideCharToMultiByte( CP_ACP, 0, string, -1, NULL, 0, NULL, NULL);
- r = IStream_Write( stm, &len, sizeof len, NULL);
+ r = IStream_Write( stm, &len, sizeof(len), NULL);
if( FAILED( r ) )
return r;
if(len == 0)
@@ -6573,10 +6573,10 @@
LPSTR str;
LPWSTR wstr;
- r = IStream_Read( stm, &len, sizeof len, &count );
+ r = IStream_Read( stm, &len, sizeof(len), &count );
if( FAILED( r ) )
return r;
- if( count != sizeof len )
+ if( count != sizeof(len) )
return E_OUTOFMEMORY;
TRACE("%ld bytes\n",len);
@@ -6633,7 +6633,7 @@
return r;
/* Write CompObj Structure to stream */
- r = IStream_Write(pstm, unknown1, sizeof unknown1, NULL);
+ r = IStream_Write(pstm, unknown1, sizeof(unknown1), NULL);
if( SUCCEEDED( r ) )
r = WriteClassStm( pstm, clsid );
@@ -6645,7 +6645,7 @@
if( SUCCEEDED( r ) )
r = STREAM_WriteString( pstm, szProgIDName );
if( SUCCEEDED( r ) )
- r = IStream_Write(pstm, unknown2, sizeof unknown2, NULL);
+ r = IStream_Write(pstm, unknown2, sizeof(unknown2), NULL);
IStream_Release( pstm );
@@ -6675,7 +6675,7 @@
for(i=0; !found; i++ )
{
- r = RegEnumKeyW( hkeyclsid, i, szKey, sizeof szKey/sizeof(WCHAR));
+ r = RegEnumKeyW( hkeyclsid, i, szKey, sizeof(szKey)/sizeof(WCHAR));
if( r != ERROR_SUCCESS )
break;
hkey = 0;
@@ -6727,7 +6727,7 @@
return E_OUTOFMEMORY;
/* get the clipboard format name */
- n = GetClipboardFormatNameW( cf, szwClipName, sizeof szwClipName );
+ n = GetClipboardFormatNameW( cf, szwClipName, sizeof(szwClipName) );
szwClipName[n]=0;
TRACE("Clipboard name is %s\n", debugstr_w(szwClipName));
@@ -6785,8 +6785,8 @@
}
/* read the various parts of the structure */
- r = IStream_Read( stm, unknown1, sizeof unknown1, &count );
- if( FAILED( r ) || ( count != sizeof unknown1 ) )
+ r = IStream_Read( stm, unknown1, sizeof(unknown1), &count );
+ if( FAILED( r ) || ( count != sizeof(unknown1) ) )
goto end;
r = ReadClassStm( stm, &clsid );
if( FAILED( r ) )
@@ -6804,8 +6804,8 @@
if( FAILED( r ) )
goto end;
- r = IStream_Read( stm, unknown2, sizeof unknown2, &count );
- if( FAILED( r ) || ( count != sizeof unknown2 ) )
+ r = IStream_Read( stm, unknown2, sizeof(unknown2), &count );
+ if( FAILED( r ) || ( count != sizeof(unknown2) ) )
goto end;
/* ok, success... now we just need to store what we found */
diff --git a/dlls/oleaut32/parsedt.c b/dlls/oleaut32/parsedt.c
index 4100764..9fc8ccc 100644
--- a/dlls/oleaut32/parsedt.c
+++ b/dlls/oleaut32/parsedt.c
@@ -260,7 +260,7 @@
{ZULU, RESERV, DTK_ZULU}, /* 00:00:00 */
};
-static unsigned int szdatetktbl = sizeof datetktbl / sizeof datetktbl[0];
+static unsigned int szdatetktbl = sizeof(datetktbl) / sizeof(datetktbl[0]);
diff --git a/dlls/serialui/confdlg.c b/dlls/serialui/confdlg.c
index 1195035..b097bf1 100644
--- a/dlls/serialui/confdlg.c
+++ b/dlls/serialui/confdlg.c
@@ -176,7 +176,7 @@
return FALSE;
}
- if(!GetWindowTextA(hControl, &lpEntry[0], sizeof lpEntry))
+ if(!GetWindowTextA(hControl, &lpEntry[0], sizeof(lpEntry)))
{
TRACE("Couldn't get window text for item %lx\n",id);
return FALSE;
@@ -349,7 +349,7 @@
if(!info)
return FALSE;
SetWindowLongA(hWnd, DWL_USER, lParam);
- snprintf(szTitle, sizeof szTitle, "Settings for %s", info->lpszDevice);
+ snprintf(szTitle, sizeof(szTitle), "Settings for %s", info->lpszDevice);
SetWindowTextA(hWnd, szTitle);
SERIALUI_DCBToDialogInfo(hWnd, info);
return TRUE;
@@ -457,7 +457,7 @@
if(r != ERROR_SUCCESS)
return FALSE;
- snprintf(szKeyName, sizeof szKeyName, "%s\\%s", lpszCommKey ,lpszDevice);
+ snprintf(szKeyName, sizeof(szKeyName), "%s\\%s", lpszCommKey ,lpszDevice);
r = RegCreateKeyA(hKeyReg, szKeyName, &hKeyPort);
if(r == ERROR_SUCCESS)
{
@@ -509,7 +509,7 @@
if(r != ERROR_SUCCESS)
return FALSE;
- snprintf(szKeyName, sizeof szKeyName, "%s\\%s", lpszCommKey ,lpszDevice);
+ snprintf(szKeyName, sizeof(szKeyName), "%s\\%s", lpszCommKey ,lpszDevice);
r = RegOpenKeyA(hKeyReg, szKeyName, &hKeyPort);
if(r == ERROR_SUCCESS)
{
diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c
index 19fa0de..2304112 100644
--- a/dlls/setupapi/devinst.c
+++ b/dlls/setupapi/devinst.c
@@ -108,7 +108,7 @@
if(info==NULL)
return FALSE;
- if(info->cbSize < sizeof *info)
+ if(info->cbSize < sizeof(*info))
return FALSE;
return FALSE;
diff --git a/dlls/shell32/shelllink.c b/dlls/shell32/shelllink.c
index dfcc2be..4d3b9f9 100644
--- a/dlls/shell32/shelllink.c
+++ b/dlls/shell32/shelllink.c
@@ -247,8 +247,8 @@
TRACE("starting %s\n",debugstr_w(buffer));
- memset(&si, 0, sizeof si);
- si.cb = sizeof si;
+ memset(&si, 0, sizeof(si));
+ si.cb = sizeof(si);
if (!CreateProcessW( NULL, buffer, NULL, NULL, FALSE, 0, NULL, NULL, &si, &pi)) return FALSE;
/* wait for a while to throttle the creation of linker processes */
@@ -404,8 +404,8 @@
TRACE("%p\n", stm);
count = 0;
- r = IStream_Read(stm, &len, sizeof len, &count);
- if ( FAILED (r) || ( count != sizeof len ) )
+ r = IStream_Read(stm, &len, sizeof(len), &count);
+ if ( FAILED (r) || ( count != sizeof(len) ) )
return E_FAIL;
if( unicode )
@@ -455,20 +455,20 @@
TRACE("%p\n",stm);
- r = IStream_Read( stm, &size, sizeof size, &count );
+ r = IStream_Read( stm, &size, sizeof(size), &count );
if( FAILED( r ) )
return r;
- if( count != sizeof loc->dwTotalSize )
+ if( count != sizeof(loc->dwTotalSize) )
return E_FAIL;
loc = HeapAlloc( GetProcessHeap(), 0, size );
if( ! loc )
return E_OUTOFMEMORY;
- r = IStream_Read( stm, &loc->dwHeaderSize, size-sizeof size, &count );
+ r = IStream_Read( stm, &loc->dwHeaderSize, size-sizeof(size), &count );
if( FAILED( r ) )
goto end;
- if( count != (size - sizeof size) )
+ if( count != (size - sizeof(size)) )
{
r = E_FAIL;
goto end;
@@ -507,13 +507,13 @@
return STG_E_INVALIDPOINTER;
dwBytesRead = 0;
- r = IStream_Read(stm, &hdr, sizeof hdr, &dwBytesRead);
+ r = IStream_Read(stm, &hdr, sizeof(hdr), &dwBytesRead);
if( FAILED( r ) )
return r;
- if( dwBytesRead != sizeof hdr)
+ if( dwBytesRead != sizeof(hdr))
return E_FAIL;
- if( hdr.dwSize != sizeof hdr)
+ if( hdr.dwSize != sizeof(hdr))
return E_FAIL;
if( !IsEqualIID(&hdr.MagicGuid, &CLSID_ShellLink) )
return E_FAIL;
@@ -604,7 +604,7 @@
DWORD count;
HRESULT r;
- r = IStream_Write( stm, &len, sizeof len, &count );
+ r = IStream_Write( stm, &len, sizeof(len), &count );
if( FAILED( r ) )
return r;
@@ -624,8 +624,8 @@
FIXME("writing empty location info\n");
- memset( &loc, 0, sizeof loc );
- loc.dwTotalSize = sizeof loc - sizeof loc.dwTotalSize;
+ memset( &loc, 0, sizeof(loc) );
+ loc.dwTotalSize = sizeof(loc) - sizeof(loc.dwTotalSize);
/* FIXME: fill this in */
@@ -658,9 +658,9 @@
This->pPidl = ILCreateFromPathW( This->sPath );
}
- memset(&header, 0, sizeof header);
- header.dwSize = sizeof header;
- memcpy(&header.MagicGuid, &CLSID_ShellLink, sizeof header.MagicGuid );
+ memset(&header, 0, sizeof(header));
+ header.dwSize = sizeof(header);
+ memcpy(&header.MagicGuid, &CLSID_ShellLink, sizeof(header.MagicGuid) );
header.wHotKey = This->wHotKey;
header.nIcon = This->iIcoNdx;
@@ -682,7 +682,7 @@
SystemTimeToFileTime ( &This->time3, &header.Time3 );
/* write the Shortcut header */
- r = IStream_Write( stm, &header, sizeof header, &count );
+ r = IStream_Write( stm, &header, sizeof(header), &count );
if( FAILED( r ) )
{
ERR("Write failed at %d\n",__LINE__);
diff --git a/dlls/shell32/shellord.c b/dlls/shell32/shellord.c
index 4b10faf..68908d2 100644
--- a/dlls/shell32/shellord.c
+++ b/dlls/shell32/shellord.c
@@ -1105,7 +1105,7 @@
TRACE("%p %d \n",cs,length);
- if( (cs == NULL) || (length < sizeof *cs) )
+ if( (cs == NULL) || (length < sizeof(*cs)) )
return FALSE;
r = RegOpenKeyW( HKEY_CURRENT_USER, szwCabLocation, &hkey );
@@ -1119,12 +1119,12 @@
}
/* if we can't read from the registry, create default values */
- if ( (r != ERROR_SUCCESS) || (cs->cLength < sizeof *cs) ||
+ if ( (r != ERROR_SUCCESS) || (cs->cLength < sizeof(*cs)) ||
(cs->cLength != length) )
{
ERR("Initializing shell cabinet settings\n");
- memset(cs, 0, sizeof *cs);
- cs->cLength = sizeof *cs;
+ memset(cs, 0, sizeof(*cs));
+ cs->cLength = sizeof(*cs);
cs->nVersion = 2;
cs->fFullPathTitle = FALSE;
cs->fSaveLocalView = TRUE;
diff --git a/dlls/user/comm16.c b/dlls/user/comm16.c
index 2548207..23c7244 100644
--- a/dlls/user/comm16.c
+++ b/dlls/user/comm16.c
@@ -332,7 +332,7 @@
OVERLAPPED ov;
DWORD count= -1;
- ZeroMemory(&ov,sizeof ov);
+ ZeroMemory(&ov,sizeof(ov));
ov.hEvent = CreateEventA(NULL,0,0,NULL);
if(ov.hEvent==INVALID_HANDLE_VALUE)
return -1;
@@ -577,7 +577,7 @@
/* set default parameters */
if(COM[port].baudrate>-1){
DCB16 dcb;
- memcpy(&dcb,&COM[port].dcb,sizeof dcb);
+ memcpy(&dcb,&COM[port].dcb,sizeof(dcb));
dcb.BaudRate=COM[port].baudrate;
/* more defaults:
* databits, parity, stopbits
@@ -923,8 +923,8 @@
return -1;
}
- memset(&dcb,0,sizeof dcb);
- dcb.DCBlength = sizeof dcb;
+ memset(&dcb,0,sizeof(dcb));
+ dcb.DCBlength = sizeof(dcb);
/*
* according to MSDN, we should first interpret lpdcb->BaudRate as follows:
diff --git a/dlls/user/tests/class.c b/dlls/user/tests/class.c
index 6deb3d7..012fdc2 100644
--- a/dlls/user/tests/class.c
+++ b/dlls/user/tests/class.c
@@ -127,7 +127,7 @@
}
/* check GetClassName */
- i = GetClassNameW(hTestWnd, str, sizeof str);
+ i = GetClassNameW(hTestWnd, str, sizeof(str));
ok(i == lstrlenW(className),
"GetClassName returned incorrect length");
ok(!lstrcmpW(className,str),
diff --git a/dlls/wininet/internet.c b/dlls/wininet/internet.c
index 9278691..8b52296 100644
--- a/dlls/wininet/internet.c
+++ b/dlls/wininet/internet.c
@@ -1569,7 +1569,7 @@
proxlen = MultiByteToWideChar( CP_ACP, 0, pi->lpszProxy, -1, NULL, 0);
prbylen= MultiByteToWideChar( CP_ACP, 0, pi->lpszProxyBypass, -1, NULL, 0);
- wlen = sizeof *piw + proxlen + prbylen;
+ wlen = sizeof(*piw) + proxlen + prbylen;
wbuffer = HeapAlloc( GetProcessHeap(), 0, wlen );
piw = (LPINTERNET_PROXY_INFOW) wbuffer;
piw->dwAccessType = pi->dwAccessType;
diff --git a/dlls/winmm/lolvldrv.c b/dlls/winmm/lolvldrv.c
index 5afd0c8..e2d65f1 100644
--- a/dlls/winmm/lolvldrv.c
+++ b/dlls/winmm/lolvldrv.c
@@ -478,7 +478,7 @@
case DRV_QUERYDRIVERIDS:
WARN("NIY call VxD\n");
/* should call VxD MMDEVLDR with (DevNode, dwParam1 and dwParam2) as pmts
- * dwParam1 is buffer and dwParam2 is sizeof buffer
+ * dwParam1 is buffer and dwParam2 is sizeof(buffer)
* I don't know where the result is stored though
*/
break;
diff --git a/dlls/winsock/socket.c b/dlls/winsock/socket.c
index 644bebc..794dc36 100644
--- a/dlls/winsock/socket.c
+++ b/dlls/winsock/socket.c
@@ -910,7 +910,7 @@
#ifdef IPX_FRAME_NONE
uipx->sipx_type=IPX_FRAME_NONE;
#endif
- memset(&uipx->sipx_zero,0,sizeof uipx->sipx_zero);
+ memset(&uipx->sipx_zero,0,sizeof(uipx->sipx_zero));
return (const struct sockaddr*)uipx;
}
#endif
diff --git a/dlls/winspool/info.c b/dlls/winspool/info.c
index 7a53610..ba150ac 100644
--- a/dlls/winspool/info.c
+++ b/dlls/winspool/info.c
@@ -2933,7 +2933,7 @@
ofs = info_size*count;
for ( i=0; i<count; i++)
{
- DWORD vallen = sizeof portname - 1;
+ DWORD vallen = sizeof(portname) - 1;
/* get the serial port values, then the printer values */
if ( i < serial_count )
diff --git a/files/smb.c b/files/smb.c
index 866650d..8d85066 100644
--- a/files/smb.c
+++ b/files/smb.c
@@ -216,7 +216,7 @@
if(fd<0)
return FALSE;
- r = setsockopt(fd, SOL_SOCKET, SO_BROADCAST, &on, sizeof on);
+ r = setsockopt(fd, SOL_SOCKET, SO_BROADCAST, &on, sizeof(on));
if(r<0)
goto err;
@@ -228,11 +228,11 @@
sin.sin_family = AF_INET;
sin.sin_port = htons(137);
- len = NB_NameReq(host,buffer,sizeof buffer);
+ len = NB_NameReq(host,buffer,sizeof(buffer));
if(len<=0)
goto err;
- r = sendto(fd, buffer, len, 0, (struct sockaddr*)&sin, sizeof sin);
+ r = sendto(fd, buffer, len, 0, (struct sockaddr*)&sin, sizeof(sin));
if(r<0)
{
FIXME("Error sending packet\n");
@@ -253,7 +253,7 @@
TRACE("Got response!\n");
fromsize = sizeof (fromaddr);
- r = recvfrom(fd, buffer, sizeof buffer, 0, (struct sockaddr*)&fromaddr, &fromsize);
+ r = recvfrom(fd, buffer, sizeof(buffer), 0, (struct sockaddr*)&fromaddr, &fromsize);
if(r<0)
goto err;
@@ -271,7 +271,7 @@
TRACE("packet is OK\n");
- memcpy(&addr->sin_addr, &buffer[58], sizeof addr->sin_addr);
+ memcpy(&addr->sin_addr, &buffer[58], sizeof(addr->sin_addr));
close(fd);
return TRUE;
@@ -525,7 +525,7 @@
TRACE("\n");
- memset(buf,0,sizeof buf);
+ memset(buf,0,sizeof(buf));
tx.buffer = buf;
tx.len = SMB_Header(tx.buffer, SMB_COM_NEGOTIATE, 0, 0);
@@ -578,7 +578,7 @@
int pcount,bcount;
struct NB_Buffer rx, tx;
- memset(buf,0,sizeof buf);
+ memset(buf,0,sizeof(buf));
tx.buffer = buf;
tx.len = SMB_Header(tx.buffer, SMB_COM_SESSION_SETUP_ANDX, 0, 0);
@@ -675,7 +675,7 @@
TRACE("%s\n",share_name);
- memset(buf,0,sizeof buf);
+ memset(buf,0,sizeof(buf));
tx.buffer = buf;
tx.len = SMB_Header(tx.buffer, SMB_COM_TREE_CONNECT, 0, user_id);
@@ -697,7 +697,7 @@
tx.buffer[tx.len++] = 0; /* null terminated password */
slen = strlen(share_name);
- if(slen<(sizeof buf-tx.len))
+ if(slen<(sizeof(buf)-tx.len))
strcpy(&tx.buffer[tx.len], share_name);
else
return FALSE;
@@ -739,7 +739,7 @@
TRACE("%s\n",filename);
- memset(buffer,0,sizeof buffer);
+ memset(buffer,0,sizeof(buffer));
len = SMB_Header(buffer, SMB_COM_NT_CREATE_ANDX, tree_id, user_id);
@@ -789,7 +789,7 @@
/* 0x39 */
SMB_ADDWORD(&buffer[len],slen); len += 2; /* size of buffer */
- if(slen<(sizeof buffer-len))
+ if(slen<(sizeof(buffer)-len))
strcpy(&buffer[len], filename);
else
return FALSE;
@@ -865,7 +865,7 @@
mode = SMB_GetMode(access,sharing);
- memset(buffer,0,sizeof buffer);
+ memset(buffer,0,sizeof(buffer));
len = SMB_Header(buffer, SMB_COM_OPEN_ANDX, tree_id, user_id);
@@ -896,7 +896,7 @@
TRACE("%s\n",filename);
- memset(buf,0,sizeof buf);
+ memset(buf,0,sizeof(buf));
tx.buffer = buf;
tx.len = SMB_Header(tx.buffer, SMB_COM_OPEN, tree_id, user_id);
@@ -1167,7 +1167,7 @@
int storagetype = 0;
int len, buf_size;
- memset(send,0,sizeof send);
+ memset(send,0,sizeof(send));
send->setup_count = 1;
send->setup = HeapAlloc(GetProcessHeap(),0,send->setup_count*2);
@@ -1216,7 +1216,7 @@
if(!SMB_SetupFindFirst(&send, filename))
return FALSE;
- memset(&recv,0,sizeof recv);
+ memset(&recv,0,sizeof(recv));
ret = SMB_Transaction2(fd, tree_id, user_id, &send, &recv);
HeapFree(GetProcessHeap(),0,send.params);
@@ -1323,7 +1323,7 @@
unsigned char *x = (unsigned char *)&sin.sin_addr;
TRACE("Connecting to %d.%d.%d.%d ...\n", x[0],x[1],x[2],x[3]);
}
- r = connect(fd, (struct sockaddr*)&sin, sizeof sin);
+ r = connect(fd, (struct sockaddr*)&sin, sizeof(sin));
if(!NB_SessionReq(fd, "*SMBSERVER", "WINE"))
{
@@ -1596,7 +1596,7 @@
if(dir->current >= dir->num_entries)
return FALSE;
- memset(data, 0, sizeof *data);
+ memset(data, 0, sizeof(*data));
ent = dir->entries[dir->current];
len = SMB_GETDWORD(&ent[0]);
@@ -1612,13 +1612,13 @@
/* copy the long filename */
fnlen = SMB_GETDWORD(&ent[0x3c]);
- if ( fnlen > (sizeof data->cFileName/sizeof(WCHAR)) )
+ if ( fnlen > (sizeof(data->cFileName)/sizeof(WCHAR)) )
return FALSE;
MultiByteToWideChar( CP_ACP, 0, &ent[0x5e], fnlen, data->cFileName,
sizeof(data->cFileName)/sizeof(WCHAR) );
/* copy the short filename */
- if ( ent[0x44] > (sizeof data->cAlternateFileName/sizeof(WCHAR)) )
+ if ( ent[0x44] > (sizeof(data->cAlternateFileName)/sizeof(WCHAR)) )
return FALSE;
MultiByteToWideChar( CP_ACP, 0, &ent[0x5e + len], ent[0x44], data->cAlternateFileName,
sizeof(data->cAlternateFileName)/sizeof(WCHAR) );
@@ -1632,7 +1632,7 @@
{
HeapFree(GetProcessHeap(),0,dir->buffer);
HeapFree(GetProcessHeap(),0,dir->entries);
- memset(dir,0,sizeof *dir);
+ memset(dir,0,sizeof(*dir));
HeapFree(GetProcessHeap(),0,dir);
return TRUE;
}
diff --git a/graphics/x11drv/xfont.c b/graphics/x11drv/xfont.c
index 04d9ca0..47e6c3c 100644
--- a/graphics/x11drv/xfont.c
+++ b/graphics/x11drv/xfont.c
@@ -1704,7 +1704,7 @@
{
BOOL bSubst;
char subsection[32];
- snprintf( subsection, sizeof subsection, "%s%i", INIAliasSection, i++ );
+ snprintf( subsection, sizeof(subsection), "%s%i", INIAliasSection, i++ );
buffer[0] = 0;
if(!RegOpenKeyA(HKEY_LOCAL_MACHINE, INIFontSection, &hkey))
@@ -2109,7 +2109,7 @@
lfd1.resolution_x = res_string;
lfd1.resolution_y = res_string;
- LFD_UnParse(buffer, sizeof buffer, &lfd1);
+ LFD_UnParse(buffer, sizeof(buffer), &lfd1);
lpstr = buffer;
}
@@ -2330,7 +2330,7 @@
for( j = i = 0, pfr = fontList; pfr; pfr = pfr->next )
{
- LFD_UnParse(buffer, sizeof buffer, pfr->resource);
+ LFD_UnParse(buffer, sizeof(buffer), pfr->resource);
i += strlen( buffer) + 1;
j += pfr->fi_count;
}
@@ -2368,7 +2368,7 @@
write( fd, &i, sizeof(int) );
for( pfr = fontList; pfr && i == j; pfr = pfr->next )
{
- LFD_UnParse(buffer, sizeof buffer, pfr->resource);
+ LFD_UnParse(buffer, sizeof(buffer), pfr->resource);
i = strlen( buffer ) + 1;
j = write( fd, buffer, i );
}
diff --git a/programs/uninstaller/main.c b/programs/uninstaller/main.c
index 365ba67..8dfd1ad 100644
--- a/programs/uninstaller/main.c
+++ b/programs/uninstaller/main.c
@@ -80,7 +80,7 @@
{ BS_PUSHBUTTON, "Exit", 0 }
};
-#define NUM (sizeof button/sizeof button[0])
+#define NUM (sizeof(button)/sizeof(button[0]))
int FetchUninstallInformation(void);
void UninstallProgram(void);
diff --git a/programs/winemenubuilder/winemenubuilder.c b/programs/winemenubuilder/winemenubuilder.c
index 7c7a6a5..9179218 100644
--- a/programs/winemenubuilder/winemenubuilder.c
+++ b/programs/winemenubuilder/winemenubuilder.c
@@ -683,7 +683,7 @@
if( !GetFullPathNameW( linkfile, MAX_PATH, filename, NULL ))
return FALSE;
- for( i=0; i<sizeof locations/sizeof locations[0]; i++ )
+ for( i=0; i<sizeof(locations)/sizeof(locations[0]); i++ )
{
if (!SHGetSpecialFolderPathW( 0, buffer, locations[i], FALSE ))
continue;
@@ -743,24 +743,24 @@
}
szWorkDir[0]=0;
- IShellLinkA_GetWorkingDirectory( sl, szWorkDir, sizeof szWorkDir);
+ IShellLinkA_GetWorkingDirectory( sl, szWorkDir, sizeof(szWorkDir));
WINE_TRACE("workdir : %s\n", szWorkDir);
szDescription[0] = 0;
- IShellLinkA_GetDescription( sl, szDescription, sizeof szDescription);
+ IShellLinkA_GetDescription( sl, szDescription, sizeof(szDescription));
WINE_TRACE("description: %s\n", szDescription);
szPath[0] = 0;
- IShellLinkA_GetPath( sl, szPath, sizeof szPath, NULL, SLGP_RAWPATH );
+ IShellLinkA_GetPath( sl, szPath, sizeof(szPath), NULL, SLGP_RAWPATH );
WINE_TRACE("path : %s\n", szPath);
szArgs[0] = 0;
- IShellLinkA_GetArguments( sl, szArgs, sizeof szArgs );
+ IShellLinkA_GetArguments( sl, szArgs, sizeof(szArgs) );
WINE_TRACE("args : %s\n", szArgs);
szIconPath[0] = 0;
IShellLinkA_GetIconLocation( sl, szIconPath,
- sizeof szIconPath, &iIconId );
+ sizeof(szIconPath), &iIconId );
WINE_TRACE("icon file : %s\n", szIconPath );
if( !szPath[0] )
@@ -963,7 +963,7 @@
{
WCHAR link[MAX_PATH];
- MultiByteToWideChar( CP_ACP, 0, token, -1, link, sizeof link );
+ MultiByteToWideChar( CP_ACP, 0, token, -1, link, sizeof(link) );
if( !Process_Link( link, bAgain ) )
{
WINE_ERR( "failed to build menu item for %s\n",token);
diff --git a/scheduler/client.c b/scheduler/client.c
index cae2c09..548de13 100644
--- a/scheduler/client.c
+++ b/scheduler/client.c
@@ -68,7 +68,7 @@
/* data structure used to pass an fd with sendmsg/recvmsg */
struct cmsg_fd
{
- int len; /* sizeof structure */
+ int len; /* size of structure */
int level; /* SOL_SOCKET */
int type; /* SCM_RIGHTS */
int fd; /* fd to pass */
diff --git a/server/named_pipe.c b/server/named_pipe.c
index aafdcbc..e595090 100644
--- a/server/named_pipe.c
+++ b/server/named_pipe.c
@@ -248,7 +248,7 @@
{
struct connect_wait *waiter;
- waiter = mem_alloc( sizeof *waiter );
+ waiter = mem_alloc( sizeof(*waiter) );
if( waiter )
{
set_waiter( &waiter->wait, func, overlapped );
diff --git a/server/registry.c b/server/registry.c
index 340c474..c2f69b0 100644
--- a/server/registry.c
+++ b/server/registry.c
@@ -1924,7 +1924,7 @@
}
else
{
- notify = (struct notify *) malloc (sizeof *notify);
+ notify = (struct notify *) malloc (sizeof(*notify));
if( notify )
{
grab_object( event );
diff --git a/server/request.c b/server/request.c
index 4d31295..969c24f 100644
--- a/server/request.c
+++ b/server/request.c
@@ -116,7 +116,7 @@
#ifndef HAVE_STRUCT_MSGHDR_MSG_ACCRIGHTS
struct cmsg_fd
{
- int len; /* sizeof structure */
+ int len; /* size of structure */
int level; /* SOL_SOCKET */
int type; /* SCM_RIGHTS */
int fd; /* fd to pass */