notepad: Use the ARRAY_SIZE() macro if we already have it.
Michael Stefaniuc
mstefani at redhat.de
Wed May 27 16:49:13 CDT 2009
Patch generated with the demos/janitorings/ARRAY_SIZE.cocci coccinelle
script from coccinelle-0.1.8.
---
programs/notepad/dialog.c | 2 +-
programs/notepad/main.c | 8 ++++----
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/programs/notepad/dialog.c b/programs/notepad/dialog.c
index 761705b..d7354e7 100644
--- a/programs/notepad/dialog.c
+++ b/programs/notepad/dialog.c
@@ -257,7 +257,7 @@ void DoOpenFile(LPCWSTR szFileName)
SetFocus(Globals.hEdit);
/* If the file starts with .LOG, add a time/date at the end and set cursor after */
- if (GetWindowTextW(Globals.hEdit, log, sizeof(log)/sizeof(log[0])) && !lstrcmpW(log, dotlog))
+ if (GetWindowTextW(Globals.hEdit, log, ARRAY_SIZE(log)) && !lstrcmpW(log, dotlog))
{
static const WCHAR lfW[] = { '\r','\n',0 };
SendMessageW(Globals.hEdit, EM_SETSEL, GetWindowTextLengthW(Globals.hEdit), -1);
diff --git a/programs/notepad/main.c b/programs/notepad/main.c
index 57018ce..c195668 100644
--- a/programs/notepad/main.c
+++ b/programs/notepad/main.c
@@ -203,10 +203,10 @@ static VOID NOTEPAD_LoadSettingFromRegistry(void)
Globals.lfFont.lfPitchAndFamily = FIXED_PITCH | FF_DONTCARE;
lstrcpyW(Globals.lfFont.lfFaceName, systemW);
- LoadStringW(Globals.hInstance, STRING_PAGESETUP_HEADERVALUE, Globals.szHeader,
- sizeof(Globals.szHeader) / sizeof(Globals.szHeader[0]));
- LoadStringW(Globals.hInstance, STRING_PAGESETUP_FOOTERVALUE, Globals.szFooter,
- sizeof(Globals.szFooter) / sizeof(Globals.szFooter[0]));
+ LoadStringW(Globals.hInstance, STRING_PAGESETUP_HEADERVALUE,
+ Globals.szHeader, ARRAY_SIZE(Globals.szHeader));
+ LoadStringW(Globals.hInstance, STRING_PAGESETUP_FOOTERVALUE,
+ Globals.szFooter, ARRAY_SIZE(Globals.szFooter));
if(RegOpenKeyW(HKEY_CURRENT_USER, notepad_reg_key, &hkey) == ERROR_SUCCESS)
{
--
1.6.3.1
More information about the wine-patches
mailing list