From 6895d635acb37ba42e53ba95ca3eee2ddbbef24d Mon Sep 17 00:00:00 2001 From: Tobias Weimer Date: Fri, 28 Nov 2014 21:19:14 +0000 Subject: Massive code cleanup and fixes for a lot of warnings reported in #837 git-svn-id: http://svn.miranda-ng.org/main/trunk@11136 1316c22d-e87f-b044-9b9b-93d7a3e3ba9c --- plugins/AVS/src/options.cpp | 2 +- .../AdvaImg/src/FreeImageToolkit/Background.cpp | 2 +- plugins/AvatarHistory/src/AvatarDlg.cpp | 2 +- plugins/BossKeyPlus/src/BossKeyIdle.cpp | 2 +- .../ChangeKeyboardLayout/src/text_operations.cpp | 6 +- plugins/DbEditorPP/src/exportimport.cpp | 9 +- plugins/DbEditorPP/src/settinglist.cpp | 4 +- plugins/HistoryStats/src/bandctrlimpl.cpp | 2 +- plugins/HistoryStats/src/dlgfilterwords.cpp | 2 +- plugins/HistoryStats/src/dlgoption_subcolumns.cpp | 4 +- plugins/HistoryStats/src/dlgoption_subglobal.cpp | 2 +- plugins/HistoryStats/src/dlgoption_subinput.cpp | 2 +- plugins/IEView/src/IEView.cpp | 2 +- plugins/KeyboardNotify/src/main.cpp | 23 ++-- plugins/KeyboardNotify/src/options.cpp | 80 ++++++------ plugins/LotusNotify/src/LotusNotify.cpp | 134 ++++++++++----------- plugins/MenuItemEx/src/main.cpp | 2 +- plugins/Msg_Export/src/FileViewer.cpp | 2 +- plugins/NewAwaySysMod/src/MsgTree.cpp | 2 +- plugins/NewAwaySysMod/src/SetAwayMsg.cpp | 2 +- plugins/OpenFolder/src/openFolder.cpp | 48 ++++---- plugins/PluginUpdater/src/DlgUpdate.cpp | 1 - plugins/Popup/src/skin.cpp | 1 - plugins/Scriver/src/input.cpp | 2 +- plugins/Scriver/src/msglog.cpp | 1 - plugins/SimpleStatusMsg/src/main.cpp | 38 +++--- plugins/SimpleStatusMsg/src/msgbox.cpp | 2 +- plugins/Spamotron/src/utils.cpp | 15 ++- plugins/TabSRMM/src/buttonsbar.cpp | 7 +- plugins/TabSRMM/src/container.cpp | 12 +- plugins/TabSRMM/src/controls.cpp | 2 +- plugins/TabSRMM/src/hotkeyhandler.cpp | 2 +- plugins/TabSRMM/src/msgdialog.cpp | 2 +- plugins/TabSRMM/src/msglog.cpp | 7 +- plugins/TabSRMM/src/sendqueue.cpp | 4 +- plugins/TabSRMM/src/templates.cpp | 3 +- plugins/TabSRMM/src/themes.cpp | 19 ++- plugins/UserInfoEx/src/Flags/svc_flagsicons.cpp | 1 - plugins/Watrack_MPD/src/main.cpp | 8 +- plugins/Weather/src/weather_ini.cpp | 2 +- plugins/WebView/src/webview_alerts.cpp | 10 +- plugins/WinterSpeak/src/DialogConfigEngine.cpp | 8 +- plugins/YARelay/src/main.cpp | 2 +- 43 files changed, 225 insertions(+), 258 deletions(-) (limited to 'plugins') diff --git a/plugins/AVS/src/options.cpp b/plugins/AVS/src/options.cpp index 7f52dc1a75..d6b94ba9a7 100644 --- a/plugins/AVS/src/options.cpp +++ b/plugins/AVS/src/options.cpp @@ -75,7 +75,7 @@ static void RemoveProtoPic(const char *szProto) if (p.szProtoname == NULL) continue; - PROTOACCOUNT* pdescr = (PROTOACCOUNT*)CallService(MS_PROTO_GETACCOUNT, 0, (LPARAM)p.szProtoname); + PROTOACCOUNT *pdescr = ProtoGetAccount(p.szProtoname); if (pdescr == NULL && lstrcmpA(p.szProtoname, szProto)) continue; diff --git a/plugins/AdvaImg/src/FreeImageToolkit/Background.cpp b/plugins/AdvaImg/src/FreeImageToolkit/Background.cpp index db481e7728..fc18667b21 100644 --- a/plugins/AdvaImg/src/FreeImageToolkit/Background.cpp +++ b/plugins/AdvaImg/src/FreeImageToolkit/Background.cpp @@ -222,6 +222,7 @@ FillBackgroundBitmap(FIBITMAP *dib, const RGBQUAD *color, int options) { return FALSE; } + RGBQUAD blend; const RGBQUAD *color_intl = color; unsigned bpp = FreeImage_GetBPP(dib); unsigned width = FreeImage_GetWidth(dib); @@ -264,7 +265,6 @@ FillBackgroundBitmap(FIBITMAP *dib, const RGBQUAD *color, int options) { bgcolor.rgbRed = src_bits[FI_RGBA_RED]; bgcolor.rgbReserved = 0xFF; } - RGBQUAD blend; GetAlphaBlendedColor(&bgcolor, color_intl, &blend); color_intl = &blend; } diff --git a/plugins/AvatarHistory/src/AvatarDlg.cpp b/plugins/AvatarHistory/src/AvatarDlg.cpp index 64ff1338c0..2c4f63afd4 100644 --- a/plugins/AvatarHistory/src/AvatarDlg.cpp +++ b/plugins/AvatarHistory/src/AvatarDlg.cpp @@ -557,9 +557,9 @@ int ShowSaveDialog(HWND hwnd, TCHAR* fn, MCONTACT hContact) ofn.lpstrFile = file; TCHAR *displayName = (TCHAR*) CallService(MS_CLIST_GETCONTACTDISPLAYNAME,hContact,GCDNF_TCHAR); + TCHAR title[MAX_PATH]; if (displayName) { - TCHAR title[MAX_PATH]; mir_sntprintf(title,sizeof(title),TranslateT("Save Avatar for %s"),displayName); ofn.lpstrTitle = title; } diff --git a/plugins/BossKeyPlus/src/BossKeyIdle.cpp b/plugins/BossKeyPlus/src/BossKeyIdle.cpp index bec29756f4..57a1044734 100644 --- a/plugins/BossKeyPlus/src/BossKeyIdle.cpp +++ b/plugins/BossKeyPlus/src/BossKeyIdle.cpp @@ -34,7 +34,7 @@ static bool IsUserIdle() { DWORD dwTick; if ( g_wMaskAdv & OPT_HIDEIFMIRIDLE ) { - CallService(MS_SYSTEM_GETIDLE, 0, (DWORD)&dwTick); + CallService(MS_SYSTEM_GETIDLE, 0, (LPARAM)&dwTick); return GetTickCount() - dwTick > (minutes * 60 * 1000); } diff --git a/plugins/ChangeKeyboardLayout/src/text_operations.cpp b/plugins/ChangeKeyboardLayout/src/text_operations.cpp index 0de0987572..b535100331 100644 --- a/plugins/ChangeKeyboardLayout/src/text_operations.cpp +++ b/plugins/ChangeKeyboardLayout/src/text_operations.cpp @@ -286,7 +286,7 @@ int ChangeLayout(HWND hTextWnd, BYTE TextOperation, BOOL CurrentWord) if (WindowType == WTYPE_RichEdit) { EditStreamData esdData; EDITSTREAM esStream = { 0 }; - esStream.dwCookie = (DWORD)&esdData; + esStream.dwCookie = (DWORD_PTR)&esdData; esStream.pfnCallback = EditStreamOutRtf; if (SendMessage(hTextWnd, EM_STREAMOUT, SF_TEXT | SF_UNICODE | SFF_SELECTION, (LPARAM)&esStream) > 0) ptszInText = GeTStringFromStreamData(&esdData); @@ -324,7 +324,7 @@ int ChangeLayout(HWND hTextWnd, BYTE TextOperation, BOOL CurrentWord) EditStreamData esdData; EDITSTREAM esStream = { 0 }; - esStream.dwCookie = (DWORD)&esdData; + esStream.dwCookie = (DWORD_PTR)&esdData; esStream.pfnCallback = EditStreamOutRtf; if (SendMessage(hTextWnd, EM_STREAMOUT, SF_TEXT | SF_UNICODE | SFF_SELECTION, (LPARAM)&esStream) != 0) ptszInText = GeTStringFromStreamData(&esdData); @@ -363,7 +363,7 @@ int ChangeLayout(HWND hTextWnd, BYTE TextOperation, BOOL CurrentWord) EditStreamData esdData; EDITSTREAM esStream = { 0 }; - esStream.dwCookie = (DWORD)&esdData; + esStream.dwCookie = (DWORD_PTR)&esdData; esStream.pfnCallback = EditStreamOutRtf; if (SendMessage(hTextWnd, EM_STREAMOUT, SF_TEXT | SF_UNICODE | SFF_SELECTION, (LPARAM)&esStream) != 0) ptszInText = GeTStringFromStreamData(&esdData); diff --git a/plugins/DbEditorPP/src/exportimport.cpp b/plugins/DbEditorPP/src/exportimport.cpp index 7bd1643e0c..2d452fac31 100644 --- a/plugins/DbEditorPP/src/exportimport.cpp +++ b/plugins/DbEditorPP/src/exportimport.cpp @@ -241,7 +241,8 @@ void exportDB(MCONTACT hContact, char *module) } else { - if (module == "") module = NULL; // reset module for all contacts export + if (*module = '\0') + module = NULL; // reset module for all contacts export } hContact = db_find_first(); @@ -422,7 +423,7 @@ void importSettings(MCONTACT hContact, char *importstring ) else if (importstring[i] == '[' && !strchr(&importstring[i+1],'='))// get the module { if (end = strpbrk(&importstring[i+1], "]")) { - if ((end+1) != '\0') *end = '\0'; + if (*(end+1) != '\0') *end = '\0'; strcpy(module, &importstring[i+1]); } } @@ -430,7 +431,7 @@ void importSettings(MCONTACT hContact, char *importstring ) !strchr(&importstring[i+2],'='))// get the module { if (end = strpbrk(&importstring[i+2], "]")) { - if ((end+1) != '\0') *end = '\0'; + if (*(end+1) != '\0') *end = '\0'; strcpy(module, &importstring[i+2]); deleteModule(module, hContact, 1); } @@ -438,7 +439,7 @@ void importSettings(MCONTACT hContact, char *importstring ) else if (strstr(&importstring[i], "=") && module[0]) // get the setting { if (end = strpbrk(&importstring[i+1], "=")) { - if ((end+1) != '\0') *end = '\0'; + if (*(end+1) != '\0') *end = '\0'; strcpy(setting, &importstring[i]); // get the type diff --git a/plugins/DbEditorPP/src/settinglist.cpp b/plugins/DbEditorPP/src/settinglist.cpp index 1cb5f3975f..c9d655c698 100644 --- a/plugins/DbEditorPP/src/settinglist.cpp +++ b/plugins/DbEditorPP/src/settinglist.cpp @@ -659,8 +659,8 @@ INT_PTR CALLBACK SettingsCompare(LPARAM lParam1, LPARAM lParam2, LPARAM myParam) const int maxSize = 1024; TCHAR text1[maxSize]; TCHAR text2[maxSize]; - ListView_GetItemText(params.hList, (int)lParam1, params.column, text1, maxSize); - ListView_GetItemText(params.hList, (int)lParam2, params.column, text2, maxSize); + ListView_GetItemText(params.hList, lParam1, params.column, text1, maxSize); + ListView_GetItemText(params.hList, lParam2, params.column, text2, maxSize); int res = _tcsicmp(text1, text2); res = (params.column == lastColumn) ? -res : res; diff --git a/plugins/HistoryStats/src/bandctrlimpl.cpp b/plugins/HistoryStats/src/bandctrlimpl.cpp index b68dadee1d..feeabb2eb1 100644 --- a/plugins/HistoryStats/src/bandctrlimpl.cpp +++ b/plugins/HistoryStats/src/bandctrlimpl.cpp @@ -124,7 +124,7 @@ LRESULT CALLBACK BandCtrlImpl::staticWndProc(HWND hWnd, UINT msg, WPARAM wParam, case BCM_SETBUTTONDATA: assert(wParam >= 1 && wParam <= pCtrl->m_Items.size()); - pCtrl->m_Items[wParam - 1].dwData = static_cast(lParam); + pCtrl->m_Items[wParam - 1].dwData = static_cast(lParam); return 0; case BCM_ISBUTTONVISIBLE: diff --git a/plugins/HistoryStats/src/dlgfilterwords.cpp b/plugins/HistoryStats/src/dlgfilterwords.cpp index cb5a0ff30d..be60d0bbf2 100644 --- a/plugins/HistoryStats/src/dlgfilterwords.cpp +++ b/plugins/HistoryStats/src/dlgfilterwords.cpp @@ -102,7 +102,7 @@ void DlgFilterWords::onWMInitDialog() { const Filter* pFilter = *i; - HANDLE hCheck = m_Sets.insertCheck(NULL, pFilter->getName().c_str(), 0, reinterpret_cast(pFilter)); + HANDLE hCheck = m_Sets.insertCheck(NULL, pFilter->getName().c_str(), 0, reinterpret_cast(pFilter)); if (m_bColProvided && m_ColFilters.find(pFilter->getID()) != m_ColFilters.end()) m_Sets.checkItem(hCheck, true); diff --git a/plugins/HistoryStats/src/dlgoption_subcolumns.cpp b/plugins/HistoryStats/src/dlgoption_subcolumns.cpp index 24cff27083..a11e1e0f52 100644 --- a/plugins/HistoryStats/src/dlgoption_subcolumns.cpp +++ b/plugins/HistoryStats/src/dlgoption_subcolumns.cpp @@ -238,7 +238,7 @@ void DlgOption::SubColumns::loadSettings() { Column* pCol = localS.getCol(j); ext::string colTitle = pCol->getTitleForOptions(); - OptionsCtrl::Check hColCheck = m_Columns.insertCheck(NULL, colTitle.c_str(), 0, reinterpret_cast(pCol)); + OptionsCtrl::Check hColCheck = m_Columns.insertCheck(NULL, colTitle.c_str(), 0, reinterpret_cast(pCol)); m_Columns.checkItem(hColCheck, pCol->isEnabled()); @@ -332,7 +332,7 @@ void DlgOption::SubColumns::addCol(int nCol) getParent()->getLocalSettings().addCol(pCol); - OptionsCtrl::Check hColCheck = m_Columns.insertCheck(NULL, pCol->getTitle(), 0, reinterpret_cast(pCol)); + OptionsCtrl::Check hColCheck = m_Columns.insertCheck(NULL, pCol->getTitle(), 0, reinterpret_cast(pCol)); m_Columns.checkItem(hColCheck, pCol->isEnabled()); diff --git a/plugins/HistoryStats/src/dlgoption_subglobal.cpp b/plugins/HistoryStats/src/dlgoption_subglobal.cpp index b89a9e1e0f..6ea0d7944c 100644 --- a/plugins/HistoryStats/src/dlgoption_subglobal.cpp +++ b/plugins/HistoryStats/src/dlgoption_subglobal.cpp @@ -249,7 +249,7 @@ void DlgOption::SubGlobal::onWMInitDialog() m_hProtocols, Protocol::getDisplayName(protoList[i]->szModuleName).c_str(), 0, - reinterpret_cast(protoList[i]->szModuleName))); + reinterpret_cast(protoList[i]->szModuleName))); } } diff --git a/plugins/HistoryStats/src/dlgoption_subinput.cpp b/plugins/HistoryStats/src/dlgoption_subinput.cpp index 7d61c56b63..d499f8d6de 100644 --- a/plugins/HistoryStats/src/dlgoption_subinput.cpp +++ b/plugins/HistoryStats/src/dlgoption_subinput.cpp @@ -111,7 +111,7 @@ void DlgOption::SubInput::onWMInitDialog() m_hProtocols, Protocol::getDisplayName(protoList[i]->szModuleName).c_str(), 0, - reinterpret_cast(protoList[i]->szModuleName))); + reinterpret_cast(protoList[i]->szModuleName))); } } diff --git a/plugins/IEView/src/IEView.cpp b/plugins/IEView/src/IEView.cpp index 07f071e3eb..a2a7a13721 100644 --- a/plugins/IEView/src/IEView.cpp +++ b/plugins/IEView/src/IEView.cpp @@ -1103,7 +1103,7 @@ BSTR IEView::getHrefFromAnchor(IHTMLElement *element) if (SUCCEEDED(element->QueryInterface(IID_IHTMLAnchorElement, (void**)&pAnchor)) && (pAnchor!=NULL)) { VARIANT variant; BSTR url; - if (SUCCEEDED(element->getAttribute(L"href", 2, &variant) && variant.vt == VT_BSTR)) { + if (SUCCEEDED(element->getAttribute(L"href", 2, &variant)) && (variant.vt == VT_BSTR)) { url = mir_tstrdup(variant.bstrVal); SysFreeString(variant.bstrVal); } diff --git a/plugins/KeyboardNotify/src/main.cpp b/plugins/KeyboardNotify/src/main.cpp index 29d448d822..73ea6cf54b 100644 --- a/plugins/KeyboardNotify/src/main.cpp +++ b/plugins/KeyboardNotify/src/main.cpp @@ -452,13 +452,13 @@ static VOID CALLBACK ReminderTimer(HWND hwnd, UINT message, UINT_PTR idEvent, DW // Support for third-party plugins and mBot's scripts -static INT_PTR EnableService(WPARAM wParam, LPARAM lParam) +static INT_PTR EnableService(WPARAM, LPARAM) { bFlashingEnabled = TRUE; return 0; } -static INT_PTR DisableService(WPARAM wParam, LPARAM lParam) +static INT_PTR DisableService(WPARAM, LPARAM) { bFlashingEnabled = FALSE; return 0; @@ -487,12 +487,12 @@ static INT_PTR EventsWereOpenedService(WPARAM wParam, LPARAM lParam) } -static INT_PTR IsFlashingActiveService(WPARAM wParam, LPARAM lParam) +static INT_PTR IsFlashingActiveService(WPARAM, LPARAM) { if (!bReminderDisabled) return 0; - return (int)getCurrentSequenceString(); + return (INT_PTR)getCurrentSequenceString(); } @@ -508,10 +508,11 @@ INT_PTR NormalizeSequenceService(WPARAM wParam, LPARAM lParam) // Support for Trigger plugin -static void ForceEventsWereOpenedThread(void *eventMaxSeconds) +static DWORD WINAPI ForceEventsWereOpenedThread(void *eventMaxSeconds) { Sleep(((WORD)eventMaxSeconds) * 1000); CallService(MS_KBDNOTIFY_EVENTSOPENED, 1, 0); + return 0; } @@ -520,7 +521,7 @@ void StartBlinkAction(char *flashSequence, WORD eventMaxSeconds) DWORD threadID = 0; if (eventMaxSeconds) - CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)ForceEventsWereOpenedThread, (void *)eventMaxSeconds, 0, &threadID); + CreateThread(NULL, 0, ForceEventsWereOpenedThread, (void *)eventMaxSeconds, 0, &threadID); CallService(MS_KBDNOTIFY_STARTBLINK, 1, (LPARAM)flashSequence); } @@ -540,7 +541,7 @@ void createProcessList(void) if (db_get_ts(NULL, KEYBDMODULE, fmtDBSettingName("process%d", i), &dbv)) ProcessList.szFileName[i] = NULL; else { - ProcessList.szFileName[i] = (TCHAR *)malloc(wcslen(dbv.ptszVal) + 1); + ProcessList.szFileName[i] = (TCHAR *)malloc((wcslen(dbv.ptszVal) + 1)*sizeof(TCHAR)); if (ProcessList.szFileName[i]) wcscpy(ProcessList.szFileName[i], dbv.ptszVal); db_free(&dbv); @@ -586,9 +587,9 @@ void LoadSettings(void) bMirandaOrWindows = db_get_b(NULL, KEYBDMODULE, "mirorwin", DEF_SETTING_MIRORWIN); wStatusMap = db_get_w(NULL, KEYBDMODULE, "status", DEF_SETTING_STATUS); wReminderCheck = db_get_w(NULL, KEYBDMODULE, "remcheck", DEF_SETTING_CHECKTIME); - bFlashLed[0] = !!db_get_b(NULL, KEYBDMODULE, "fnum", DEF_SETTING_FLASHNUM); - bFlashLed[1] = !!db_get_b(NULL, KEYBDMODULE, "fcaps", DEF_SETTING_FLASHCAPS); - bFlashLed[2] = !!db_get_b(NULL, KEYBDMODULE, "fscroll", DEF_SETTING_FLASHSCROLL); + bFlashLed[0] = db_get_b(NULL, KEYBDMODULE, "fnum", DEF_SETTING_FLASHNUM); + bFlashLed[1] = db_get_b(NULL, KEYBDMODULE, "fcaps", DEF_SETTING_FLASHCAPS); + bFlashLed[2] = db_get_b(NULL, KEYBDMODULE, "fscroll", DEF_SETTING_FLASHSCROLL); bFlashEffect = db_get_b(NULL, KEYBDMODULE, "feffect", DEF_SETTING_FLASHEFFECT); bSequenceOrder = db_get_b(NULL, KEYBDMODULE, "order", DEF_SETTING_SEQORDER); wCustomTheme = db_get_w(NULL, KEYBDMODULE, "custom", DEF_SETTING_CUSTOMTHEME); @@ -729,7 +730,7 @@ static int OnMetaChanged(WPARAM wParam, LPARAM) return 0; } -static int ModulesLoaded(WPARAM wParam, LPARAM lParam) +static int ModulesLoaded(WPARAM, LPARAM) { TCHAR eventPrefix[MAX_PATH+1], eventName[MAX_PATH+1]; diff --git a/plugins/KeyboardNotify/src/options.cpp b/plugins/KeyboardNotify/src/options.cpp index 99c459e453..63df423bd6 100644 --- a/plugins/KeyboardNotify/src/options.cpp +++ b/plugins/KeyboardNotify/src/options.cpp @@ -87,7 +87,7 @@ BYTE trillianLedsMsg, trillianLedsURL, trillianLedsFile, trillianLedsOther; // ** // ** Initialize the Miranda options page // ** -int InitializeOptions(WPARAM wParam,LPARAM lParam) +int InitializeOptions(WPARAM wParam,LPARAM) { OPTIONSDIALOGPAGE odp = { sizeof(odp) }; odp.hInstance = hInst; @@ -654,17 +654,14 @@ INT_PTR CALLBACK DlgProcEffectOptions(HWND hwndDlg, UINT msg, WPARAM wParam, LPA if(IsDlgButtonChecked(hwndDlg, IDC_INTURN) == BST_CHECKED) db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_INTURN); + else if (IsDlgButtonChecked(hwndDlg, IDC_INSEQUENCE) == BST_CHECKED) + db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_INSEQUENCE); + else if (IsDlgButtonChecked(hwndDlg, IDC_CUSTOM) == BST_CHECKED) + db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_CUSTOM); + else if (IsDlgButtonChecked(hwndDlg, IDC_TRILLIAN) == BST_CHECKED) + db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_TRILLIAN); else - if (IsDlgButtonChecked(hwndDlg, IDC_INSEQUENCE) == BST_CHECKED) - db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_INSEQUENCE); - else - if (IsDlgButtonChecked(hwndDlg, IDC_CUSTOM) == BST_CHECKED) - db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_CUSTOM); - else - if (IsDlgButtonChecked(hwndDlg, IDC_TRILLIAN) == BST_CHECKED) - db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_TRILLIAN); - else - db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_SAMETIME); + db_set_b(NULL, KEYBDMODULE, "feffect", FLASH_SAMETIME); db_set_b(NULL, KEYBDMODULE, "order", (BYTE)SendDlgItemMessage(hwndDlg, IDC_SEQORDER, CB_GETITEMDATA, (WPARAM)SendDlgItemMessage(hwndDlg, IDC_SEQORDER, CB_GETCURSEL, 0, 0), 0)); db_set_w(NULL, KEYBDMODULE, "custom", (WORD)SendDlgItemMessage(hwndDlg, IDC_SCUSTOM, CB_GETITEMDATA, (WPARAM)SendDlgItemMessage(hwndDlg, IDC_SCUSTOM, CB_GETCURSEL, 0, 0), 0)); @@ -715,7 +712,7 @@ INT_PTR CALLBACK DlgProcThemeOptions(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR int index = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_INSERTSTRING, (WPARAM)-1, (LPARAM)dbv.ptszVal); db_free(&dbv); if (index != CB_ERR && index != CB_ERRSPACE) { - str = (TCHAR *)malloc(MAX_PATH+1); + str = (TCHAR *)malloc((MAX_PATH+1)*sizeof(TCHAR)); if (str) if (db_get_ts(NULL, KEYBDMODULE, fmtDBSettingName("custom%d", i), &dbv)) str[0] = _T('\0'); @@ -782,11 +779,11 @@ INT_PTR CALLBACK DlgProcThemeOptions(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR return TRUE; case IDC_CUSTOMSTRING: if(HIWORD(wParam) == EN_CHANGE) { - int item; TCHAR theme[MAX_PATH+1], customAux[MAX_PATH+1]; GetDlgItemText(hwndDlg, IDC_THEME, theme, SIZEOF(theme)); - if ((item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme)) == CB_ERR) + int item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme); + if (item == CB_ERR) return TRUE; str = (TCHAR *)SendDlgItemMessage(hwndDlg, IDC_THEME, CB_GETITEMDATA, (WPARAM)item, 0); if (str) { @@ -809,14 +806,13 @@ INT_PTR CALLBACK DlgProcThemeOptions(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR return TRUE; case IDC_ADD: { - int item; TCHAR theme[MAX_PATH+1]; GetDlgItemText(hwndDlg, IDC_THEME, theme, SIZEOF(theme)); if (!theme[0]) return TRUE; - item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_ADDSTRING, 0, (LPARAM)theme); - str = (TCHAR *)malloc(MAX_PATH+1); + int item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_ADDSTRING, 0, (LPARAM)theme); + str = (TCHAR *)malloc((MAX_PATH+1)*sizeof(TCHAR)); if (str) { GetDlgItemText(hwndDlg, IDC_CUSTOMSTRING, str, MAX_PATH); SetDlgItemText(hwndDlg, IDC_CUSTOMSTRING, normalizeCustomString(str)); @@ -830,11 +826,10 @@ INT_PTR CALLBACK DlgProcThemeOptions(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR return TRUE; case IDC_UPDATE: { - int item; TCHAR theme[MAX_PATH+1]; GetDlgItemText(hwndDlg, IDC_THEME, theme, SIZEOF(theme)); - item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme); + int item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme); str = (TCHAR *)SendDlgItemMessage(hwndDlg, IDC_THEME, CB_GETITEMDATA, (WPARAM)item, 0); if (str) { GetDlgItemText(hwndDlg, IDC_CUSTOMSTRING, str, MAX_PATH); @@ -846,11 +841,10 @@ INT_PTR CALLBACK DlgProcThemeOptions(HWND hwndDlg, UINT msg, WPARAM wParam, LPAR return TRUE; case IDC_DELETE: { - int item; TCHAR theme[MAX_PATH+1]; GetDlgItemText(hwndDlg, IDC_THEME, theme, SIZEOF(theme)); - item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme); + int item = SendDlgItemMessage(hwndDlg, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme); str = (TCHAR *)SendDlgItemMessage(hwndDlg, IDC_THEME, CB_GETITEMDATA, (WPARAM)item, 0); if (str) free(str); @@ -1033,14 +1027,14 @@ void exportThemes(const TCHAR *filename) void importThemes(const TCHAR *filename, BOOL overrideExisting) { - int status=0; - size_t i; FILE *fImport = _tfopen(filename, _T("rt")); - TCHAR buffer[MAX_PATH+1], theme[MAX_PATH+1], *str; - if (!fImport) return; + int status=0; + size_t i; + TCHAR buffer[MAX_PATH+1], theme[MAX_PATH+1], *str; + while (_fgetts(buffer, MAX_PATH, fImport) != NULL) { for (str=buffer; *str && isspace(*str); str++); //ltrim if (!*str || *str == ';') //empty line or comment @@ -1067,13 +1061,12 @@ void importThemes(const TCHAR *filename, BOOL overrideExisting) void writeThemeToCombo(const TCHAR *theme, const TCHAR *custom, BOOL overrideExisting) { - int item; TCHAR *str; - item = SendDlgItemMessage(hwndTheme, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme); + int item = SendDlgItemMessage(hwndTheme, IDC_THEME, CB_FINDSTRINGEXACT, -1, (LPARAM)theme); if (item == CB_ERR) { item = SendDlgItemMessage(hwndTheme, IDC_THEME, CB_ADDSTRING, 0, (LPARAM)theme); - str = (TCHAR *)malloc(MAX_PATH+1); + str = (TCHAR *)malloc((MAX_PATH+1)*sizeof(TCHAR)); if (str) _tcscpy(str, custom); SendDlgItemMessage(hwndTheme, IDC_THEME, CB_SETITEMDATA, (WPARAM)item, (LPARAM)str); @@ -1174,7 +1167,7 @@ INT_PTR CALLBACK DlgProcProcesses(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM TCHAR szFileNameAux[MAX_PATH+1]; SendDlgItemMessage(hwndDlg, IDC_PROGRAMS, CB_GETLBTEXT, (WPARAM)i, (LPARAM)szFileNameAux); - ProcessListAux.szFileName[i] = (TCHAR *)malloc(_tcslen(szFileNameAux) + 1); + ProcessListAux.szFileName[i] = (TCHAR *)malloc((_tcslen(szFileNameAux) + 1)*sizeof(TCHAR)); if (ProcessListAux.szFileName[i]) _tcscpy(ProcessListAux.szFileName[i], szFileNameAux); } @@ -1193,7 +1186,7 @@ INT_PTR CALLBACK DlgProcProcesses(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM void createProcessListAux(void) { ProcessListAux.count = ProcessList.count; - ProcessListAux.szFileName = (TCHAR **)malloc(ProcessListAux.count * sizeof(char *)); + ProcessListAux.szFileName = (TCHAR **)malloc(ProcessListAux.count * sizeof(TCHAR *)); if (!ProcessListAux.szFileName) ProcessListAux.count = 0; else @@ -1201,7 +1194,7 @@ void createProcessListAux(void) if (!ProcessList.szFileName[i]) ProcessListAux.szFileName[i] = NULL; else { - ProcessListAux.szFileName[i] = (TCHAR *)malloc(_tcslen(ProcessList.szFileName[i]) + 1); + ProcessListAux.szFileName[i] = (TCHAR *)malloc((_tcslen(ProcessList.szFileName[i]) + 1)*sizeof(TCHAR)); if (ProcessListAux.szFileName[i]) _tcscpy(ProcessListAux.szFileName[i], ProcessList.szFileName[i]); } @@ -1337,9 +1330,6 @@ INT_PTR CALLBACK DlgProcXstatusList(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA { WPARAM j; - int imageCount; - HICON hIconAux; - HIMAGELIST hImageList; TVINSERTSTRUCT tvis={0}; TVITEM tvi={0}; HTREEITEM hSectionItem, hItem; @@ -1351,16 +1341,19 @@ INT_PTR CALLBACK DlgProcXstatusList(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA if (!XstatusListAux) return TRUE; // Calculate hImageList size - imageCount=1; + int imageCount=1; for (int i = 0; i < ProtoList.protoCount; i++) if (ProtoList.protoInfo[i].enabled && XstatusListAux[i].count) imageCount += XstatusListAux[i].count; - hImageList = ImageList_Create(GetSystemMetrics(SM_CXSMICON), GetSystemMetrics(SM_CYSMICON), ILC_COLOR32 | ILC_MASK, imageCount, imageCount); + HIMAGELIST hImageList = ImageList_Create(GetSystemMetrics(SM_CXSMICON), GetSystemMetrics(SM_CYSMICON), ILC_COLOR32 | ILC_MASK, imageCount, imageCount); TreeView_SetImageList(hwndTree, hImageList, TVSIL_NORMAL); - ImageList_AddIcon(hImageList, hIconAux=(HICON)LoadImage(GetModuleHandle(NULL), MAKEINTRESOURCE(IDI_BLANK), IMAGE_ICON, 0, 0, 0)); - if (hIconAux) DestroyIcon(hIconAux); + HICON hIconAux=(HICON)LoadImage(GetModuleHandle(NULL), MAKEINTRESOURCE(IDI_BLANK), IMAGE_ICON, 0, 0, 0); + if (hIconAux) { + ImageList_AddIcon(hImageList, hIconAux); + DestroyIcon(hIconAux); + } TreeView_SelectItem(hwndTree, NULL); ShowWindow(hwndTree, SW_HIDE); @@ -1368,9 +1361,8 @@ INT_PTR CALLBACK DlgProcXstatusList(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA for (int i = 0; i < ProtoList.protoCount; i++) if (ProtoList.protoInfo[i].enabled && XstatusListAux[i].count) { - HTREEITEM hParent; - - int count; PROTOACCOUNT** protos; + int count; + PROTOACCOUNT **protos; ProtoEnumAccounts( &count, &protos ); tvis.hParent = NULL; @@ -1382,7 +1374,7 @@ INT_PTR CALLBACK DlgProcXstatusList(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA tvis.item.state = TVIS_BOLD|TVIS_EXPANDED; tvis.item.iImage = tvis.item.iSelectedImage = ImageList_AddIcon(hImageList, hIconAux=(HICON)CallProtoService(ProtoList.protoInfo[i].szProto, PS_LOADICON, PLI_PROTOCOL, 0)); if (hIconAux) DestroyIcon(hIconAux); - hParent = TreeView_InsertItem(hwndTree, &tvis); + HTREEITEM hParent = TreeView_InsertItem(hwndTree, &tvis); for (j = 0; j < XstatusListAux[i].count; j++) { tvis.hParent = hParent; tvis.item.mask = TVIF_TEXT|TVIF_PARAM|TVIF_IMAGE|TVIF_SELECTEDIMAGE; @@ -1426,10 +1418,8 @@ INT_PTR CALLBACK DlgProcXstatusList(HWND hwndDlg, UINT msg, WPARAM wParam, LPARA case WM_DESTROY: { - HIMAGELIST hImageList; - // Destroy tree view imagelist since it does not get destroyed automatically (see msdn docs) - hImageList = TreeView_GetImageList(GetDlgItem(hwndDlg, IDC_TREE_XSTATUS), TVSIL_STATE); + HIMAGELIST hImageList = TreeView_GetImageList(GetDlgItem(hwndDlg, IDC_TREE_XSTATUS), TVSIL_STATE); if (hImageList) { TreeView_SetImageList(GetDlgItem(hwndDlg, IDC_TREE_XSTATUS), NULL, TVSIL_STATE); ImageList_Destroy(hImageList); diff --git a/plugins/LotusNotify/src/LotusNotify.cpp b/plugins/LotusNotify/src/LotusNotify.cpp index fd18559cf1..4898650d87 100644 --- a/plugins/LotusNotify/src/LotusNotify.cpp +++ b/plugins/LotusNotify/src/LotusNotify.cpp @@ -31,7 +31,7 @@ boolean volatile Plugin_Terminated = false; CRITICAL_SECTION checkthreadCS; HANDLE hMenuService = NULL; -HANDLE hMenuHandle = NULL; +HGENMENU hMenuHandle = NULL; HANDLE hCheckEvent = NULL; HANDLE hCheckHook = NULL; HANDLE hHookModulesLoaded = NULL; @@ -127,8 +127,7 @@ STATUS LNPUBLIC __stdcall EMCallBack (EMRECORD * pData) //It don't work (don't know why), and Mirandas Load function is called with value 1 or 0 as parameter... __declspec(dllexport)STATUS LNPUBLIC MainEntryPoint (void) { - STATUS rc; - rc = EMRegister1 (EM_GETPASSWORD, EM_REG_BEFORE | EM_REG_AFTER, EMCallBack, 0, &hLotusRegister); //Extension Manager must know that we are here + STATUS rc = EMRegister1 (EM_GETPASSWORD, EM_REG_BEFORE | EM_REG_AFTER, EMCallBack, 0, &hLotusRegister); //Extension Manager must know that we are here if(rc) { //Extension magager don't know who we are :( startuperror+=8; @@ -227,12 +226,12 @@ void init_pluginname() { char text[MAX_PATH], *p, *q; WIN32_FIND_DATAA ffd; - HANDLE hFind; // Try to find name of the file having original letter sizes GetModuleFileNameA(hInst, text, sizeof(text)); - if((hFind = FindFirstFileA(text, &ffd)) != INVALID_HANDLE_VALUE) { + HANDLE hFind = FindFirstFileA(text, &ffd); + if(hFind != INVALID_HANDLE_VALUE) { strncpy_s(text, SIZEOF(text), ffd.cFileName, strlen(ffd.cFileName)); FindClose(hFind); } @@ -319,8 +318,7 @@ BOOL checkFilters(TCHAR* str, int field) //subfunction called from popup plugin callback function void Click(HWND hWnd,BOOL execute) { - POPUPATT *pid=NULL; - pid = (POPUPATT*)CallService(MS_POPUP_GETPLUGINDATA, (WPARAM)hWnd,(LPARAM)pid); + POPUPATT *pid = (POPUPATT*)PUGetPluginData(hWnd); if(settingOnceOnly&&settingNonClickedOnly) (getEl(pid->id))->clicked=TRUE;//add to msgs list @@ -370,8 +368,7 @@ static LRESULT CALLBACK PopupDlgProc(HWND hWnd, UINT message, WPARAM wParam, LPA case UM_INITPOPUP: { - POPUPATT *pid=NULL; - pid = (POPUPATT*)CallService(MS_POPUP_GETPLUGINDATA, (WPARAM)hWnd,(LPARAM)pid); + POPUPATT *pid = (POPUPATT*)PUGetPluginData(hWnd); addPopup(pid->id,hWnd); //PUDeletePopUp(hWnd); break; @@ -379,8 +376,7 @@ static LRESULT CALLBACK PopupDlgProc(HWND hWnd, UINT message, WPARAM wParam, LPA case UM_FREEPLUGINDATA: { - POPUPATT * mpd = NULL; - mpd = (POPUPATT*)CallService(MS_POPUP_GETPLUGINDATA, (WPARAM)hWnd,(LPARAM)mpd); + POPUPATT *mpd = (POPUPATT*)PUGetPluginData(hWnd); if (mpd > 0) free(mpd); return TRUE; //TRUE or FALSE is the same, it gets ignored. } @@ -537,7 +533,7 @@ void ErMsgByLotusCode(STATUS erno) //set menu avainability -static void LNEnableMenuItem(HANDLE hMenuItem, BOOL bEnable) +static void LNEnableMenuItem(HGENMENU hMenuItem, BOOL bEnable) { log_p(L"LNEnableMenuItem: bEnable=%d", bEnable); CLISTMENUITEM clmi = {0}; @@ -546,7 +542,7 @@ static void LNEnableMenuItem(HANDLE hMenuItem, BOOL bEnable) if ( !bEnable ) clmi.flags |= CMIF_GRAYED; - CallService(MS_CLIST_MODIFYMENUITEM, (WPARAM)hMenuItem, (LPARAM)&clmi); + Menu_ModifyItem(hMenuItem, &clmi); } @@ -755,10 +751,9 @@ void checkthread(void*) field_copy_UNICODE[field_len] = '\0'; - WCHAR msgFrom[512]; - WCHAR msgSubject[512]; - ZeroMemory(msgFrom,512); - ZeroMemory(msgSubject,512); + WCHAR msgFrom[512], msgSubject[512]; + ZeroMemory(msgFrom,sizeof(msgFrom)); + ZeroMemory(msgSubject,sizeof(msgSubject)); if(wcslen(field_from_UNICODE) < 512 && wcslen(field_from_UNICODE) > 3 && wcsstr(field_from_UNICODE, L"CN=") == field_from_UNICODE) _tcsncpy_s(msgFrom, &(field_from_UNICODE[3]), wcscspn(field_from_UNICODE, L"/")-3 ); @@ -857,7 +852,7 @@ errorblock: //hooked notification from service that listning to check lotus -static int eventCheck(WPARAM wParam,LPARAM lParam) +static int eventCheck(WPARAM,LPARAM) { log(L"check event..."); check(); @@ -874,7 +869,7 @@ INT_PTR PluginMenuCommand(WPARAM wParam, LPARAM lParam) //window timer callback function, called on timer event -VOID CALLBACK atTime(HWND hWnd, UINT uMsg, UINT idEvent, DWORD dwTime) +void CALLBACK atTime(HWND hWnd, UINT uMsg, UINT_PTR idEvent, DWORD dwTime) { log(L"atTime: start"); BOOL b = KillTimer(hTimerWnd, idEvent); @@ -883,7 +878,7 @@ VOID CALLBACK atTime(HWND hWnd, UINT uMsg, UINT idEvent, DWORD dwTime) check(); if(settingInterval != 0){ log_p(L"atTime: SetTimer settingInterval=%d", settingInterval * 60000); - SetTimer(hTimerWnd, TID, settingInterval * 60000, (TIMERPROC)atTime); + SetTimer(hTimerWnd, TID, settingInterval * 60000, atTime); } } } @@ -965,44 +960,49 @@ void lookupLotusDefaultSettings(HWND hwndDlg) //get variables values stored in db. void LoadSettings() { - DBVARIANT dbv; - char buff[128]; - int i=0; - settingInterval = (INT)db_get_dw(NULL, PLUGINNAME, "LNInterval", 15); settingInterval1 = (INT)db_get_dw(NULL, PLUGINNAME, "LNInterval1", 0); //if(!db_get(NULL, PLUGINNAME, "LNInterval1",&dbv)) // settingInterval1 = int(dbv.lVal); //db_free(&dbv); //settingInterval=dbv.lVal; - if(!db_get_s(NULL, PLUGINNAME, "LNDatabase", &dbv)) + if(!db_get_s(NULL, PLUGINNAME, "LNDatabase", &dbv)){ strncpy_s(settingDatabase, _countof(settingDatabase), dbv.pszVal, SIZEOF(settingDatabase)); - db_free(&dbv); - if(!db_get_s(NULL, PLUGINNAME, "LNServer", &dbv)) + db_free(&dbv); + } + if(!db_get_s(NULL, PLUGINNAME, "LNServer", &dbv)) { strncpy_s(settingServer, _countof(settingServer), dbv.pszVal, SIZEOF(settingServer)); - db_free(&dbv); - if(!db_get_s(NULL, PLUGINNAME, "LNServerSec", &dbv)) + db_free(&dbv); + } + if(!db_get_s(NULL, PLUGINNAME, "LNServerSec", &dbv)) { strncpy_s(settingServerSec, _countof(settingServerSec), dbv.pszVal, SIZEOF(settingServerSec)); - db_free(&dbv); - if(!db_get(NULL, PLUGINNAME, "LNPassword", &dbv)) + db_free(&dbv); + } + if(!db_get(NULL, PLUGINNAME, "LNPassword", &dbv)) { strncpy_s(settingPassword, _countof(settingPassword), dbv.pszVal, SIZEOF(settingPassword)); - db_free(&dbv); - if(!db_get_s(NULL, PLUGINNAME, "LNCommand", &dbv, DBVT_ASCIIZ)) + db_free(&dbv); + } + if(!db_get_s(NULL, PLUGINNAME, "LNCommand", &dbv, DBVT_ASCIIZ)) { strncpy_s(settingCommand, _countof(settingCommand), dbv.pszVal, SIZEOF(settingCommand)); - db_free(&dbv); - if(!db_get_s(NULL, PLUGINNAME, "LNParameters", &dbv, DBVT_ASCIIZ)) + db_free(&dbv); + } + if(!db_get_s(NULL, PLUGINNAME, "LNParameters", &dbv, DBVT_ASCIIZ)) { strncpy_s(settingParameters, _countof(settingParameters), dbv.pszVal, SIZEOF(settingParameters)); - db_free(&dbv); + db_free(&dbv); + } - if(!db_get_ts(NULL, PLUGINNAME, "LNFilterSender",&dbv)) + if(!db_get_ts(NULL, PLUGINNAME, "LNFilterSender",&dbv)) { _tcsncpy_s(settingFilterSender, dbv.ptszVal, _TRUNCATE); - db_free(&dbv); - if(!db_get_ts(NULL, PLUGINNAME, "LNFilterSubject",&dbv)) + db_free(&dbv); + } + if(!db_get_ts(NULL, PLUGINNAME, "LNFilterSubject",&dbv)) { _tcsncpy_s(settingFilterSubject, dbv.ptszVal, _TRUNCATE); - db_free(&dbv); - if(!db_get_ts(NULL, PLUGINNAME, "LNFilterTo",&dbv)) + db_free(&dbv); + } + if(!db_get_ts(NULL, PLUGINNAME, "LNFilterTo",&dbv)) { _tcsncpy_s(settingFilterTo, dbv.ptszVal, _TRUNCATE); - db_free(&dbv); + db_free(&dbv); + } settingOnceOnly = db_get_b (NULL, PLUGINNAME, "LNOnceOnly",0); @@ -1015,8 +1015,10 @@ void LoadSettings() settingNewestID = (DWORD)db_get_dw(NULL, PLUGINNAME, "LNNewestID", 0); settingIniAnswer = db_get_b(NULL, PLUGINNAME, "LNIniAnswer", 0); settingIniCheck = db_get_b(NULL, PLUGINNAME, "LNIniCheck", 0); - - for(i = 0; i < STATUS_COUNT; i++) { + + DBVARIANT dbv; + for(int i = 0; i < STATUS_COUNT; i++) { + char buff[128]; mir_snprintf(buff, SIZEOF(buff), "LNStatus%d", i); settingStatus[i] = (db_get_b(0, PLUGINNAME, buff, 0) == 1); } @@ -1142,7 +1144,7 @@ INT_PTR CALLBACK DlgProcLotusNotifyOpts(HWND hwndDlg, UINT msg, WPARAM wParam, L int i; char text[MAXENVVALUE]; i=SendDlgItemMessage(hwndDlg,IDC_SERVER,CB_GETCURSEL,0,0); - SendDlgItemMessageA(hwndDlg,IDC_SERVER,CB_GETLBTEXT,(WPARAM)i,(LONG)/* (LPSTR) */text); + SendDlgItemMessageA(hwndDlg,IDC_SERVER,CB_GETLBTEXT,(WPARAM)i,(LPARAM)text); SetDlgItemTextA(hwndDlg,IDC_SERVER,text); break; } @@ -1154,7 +1156,7 @@ INT_PTR CALLBACK DlgProcLotusNotifyOpts(HWND hwndDlg, UINT msg, WPARAM wParam, L SendDlgItemMessage(hwndDlg, IDC_SERVER, CB_RESETCONTENT ,0, 0); fillServersList(hwndDlg); SendDlgItemMessageA(hwndDlg, IDC_SERVER, CB_ADDSTRING, 0, (LPARAM)settingServer); - SendDlgItemMessageA(hwndDlg, IDC_SERVER, CB_SELECTSTRING , -1, (LPARAM)(LPCSTR)settingServer); + SendDlgItemMessageA(hwndDlg, IDC_SERVER, CB_SELECTSTRING , -1, (LPARAM)settingServer); break; } } @@ -1263,7 +1265,6 @@ INT_PTR CALLBACK DlgProcLotusNotifyOpts(HWND hwndDlg, UINT msg, WPARAM wParam, L { char buff[128]; - int i=0; GetDlgItemTextA(hwndDlg, IDC_SERVER, settingServer, sizeof(settingServer)); db_set_s(NULL, PLUGINNAME, "LNServer", settingServer ); db_set_s(NULL, PLUGINNAME, "LNServerSec", settingServerSec); @@ -1283,34 +1284,34 @@ INT_PTR CALLBACK DlgProcLotusNotifyOpts(HWND hwndDlg, UINT msg, WPARAM wParam, L db_set_b(NULL, PLUGINNAME, "LNIniCheck", settingIniCheck); db_set_b(NULL, PLUGINNAME, "LNIniAnswer", settingIniAnswer); - for(i = 0; i < STATUS_COUNT ; i++){ + for(int i = 0; i < STATUS_COUNT ; i++){ mir_snprintf(buff, SIZEOF(buff), "LNStatus%d", i); settingStatus[i] = (ListView_GetCheckState(GetDlgItem(hwndDlg, IDC_STATUS), i) ? TRUE : FALSE); db_set_b(0, PLUGINNAME, buff, settingStatus[i] ? 1 : 0); } settingFilterSender[0] = 0; - for(i=0; ihItemToOrder(TreeView_GetNextItem(hTreeView, TreeItem ? TreeItem->hItem : NULL, TVFlag)); - if (Order =- -1) + if (Order == -1) return NULL; return (Order <= TREECTRL_ROOTORDEROFFS) ? (CBaseTreeItem*)&TreeCtrl->RootItems[ROOT_ORDER_TO_INDEX(Order)] : (CBaseTreeItem*)&TreeCtrl->Value[Order]; diff --git a/plugins/NewAwaySysMod/src/SetAwayMsg.cpp b/plugins/NewAwaySysMod/src/SetAwayMsg.cpp index 8fbd79cd69..12cd708ddc 100644 --- a/plugins/NewAwaySysMod/src/SetAwayMsg.cpp +++ b/plugins/NewAwaySysMod/src/SetAwayMsg.cpp @@ -613,7 +613,7 @@ INT_PTR CALLBACK SetAwayMsgDlgProc(HWND hwndDlg, UINT uMsg, WPARAM wParam, LPARA ti.cbSize = sizeof(ti); ti.uFlags = TTF_IDISHWND | TTF_SUBCLASS; for (int i = 0; i < SIZEOF(Tooltips); i++) { - ti.uId = (UINT)GetDlgItem(hwndDlg, Tooltips[i].DlgItemID); + ti.uId = (UINT_PTR)GetDlgItem(hwndDlg, Tooltips[i].DlgItemID); ti.lpszText = TranslateTS(Tooltips[i].Text); SendMessage(hWndTooltips, TTM_ADDTOOL, 0, (LPARAM)&ti); } diff --git a/plugins/OpenFolder/src/openFolder.cpp b/plugins/OpenFolder/src/openFolder.cpp index 631102fd26..e8860cecd7 100644 --- a/plugins/OpenFolder/src/openFolder.cpp +++ b/plugins/OpenFolder/src/openFolder.cpp @@ -27,13 +27,13 @@ BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved) return TRUE; } -static INT_PTR MenuCommand_OpenFolder(WPARAM wParam, LPARAM lParam) +static INT_PTR MenuCommand_OpenFolder(WPARAM, LPARAM) { TCHAR szMirandaPath[MAX_PATH]; GetModuleFileName( GetModuleHandle(NULL), szMirandaPath, SIZEOF(szMirandaPath)); TCHAR* p = _tcsrchr( szMirandaPath, '\\' ); - if ( p && p + 1 ) - *( p + 1 ) = 0; + if ( p && *(++p)) + *p = 0; if ( GetAsyncKeyState( VK_CONTROL ) & 0x8000 ) ShellExecute(0, _T("explore"), szMirandaPath, 0, 0, SW_SHOWNORMAL); @@ -44,7 +44,7 @@ static INT_PTR MenuCommand_OpenFolder(WPARAM wParam, LPARAM lParam) } // toptoolbar (if plugin is installed) -static int ToptoolBarHook(WPARAM wParam, LPARAM lParam) +static int ToptoolBarHook(WPARAM, LPARAM) { TTBButton ttb = { sizeof(ttb) }; ttb.hIconHandleUp = icon.hIcolib; @@ -55,28 +55,9 @@ static int ToptoolBarHook(WPARAM wParam, LPARAM lParam) return 0; } -static int ModulesLoaded(WPARAM wParam, LPARAM lParam) +static int ModulesLoaded(WPARAM, LPARAM) { HookEvent(ME_TTB_MODULELOADED, ToptoolBarHook); - - // hotkeys service (0.8+) - HOTKEYDESC hotkey = { 0 }; - hotkey.cbSize = sizeof(hotkey); - hotkey.dwFlags = HKD_TCHAR; - hotkey.pszName = "Open Folder"; - hotkey.ptszDescription = LPGENT("Open Folder"); - hotkey.ptszSection = LPGENT("Main"); - hotkey.pszService = MS_OPENFOLDER_OPEN; - hotkey.DefHotKey = MAKEWORD( 'O', HOTKEYF_SHIFT | HOTKEYF_ALT ); - Hotkey_Register(&hotkey); - - CLISTMENUITEM mi = { sizeof(mi) }; - mi.position = 0x7FFFFFFF; - mi.flags = CMIF_TCHAR; - mi.icolibItem = icon.hIcolib; - mi.ptszName = LPGENT("Open Folder"); - mi.pszService = MS_OPENFOLDER_OPEN; - Menu_AddMainMenuItem(&mi); return 0; } @@ -104,6 +85,25 @@ extern "C" int __declspec( dllexport ) Load() // icolib (0.7+) Icon_Register(hInst, LPGEN("Open Folder"), &icon, 1, OPENFOLDER_MODULE_NAME); + + // hotkeys service (0.8+) + HOTKEYDESC hotkey = { 0 }; + hotkey.cbSize = sizeof(hotkey); + hotkey.dwFlags = HKD_TCHAR; + hotkey.pszName = "Open Folder"; + hotkey.ptszDescription = LPGENT("Open Folder"); + hotkey.ptszSection = LPGENT("Main"); + hotkey.pszService = MS_OPENFOLDER_OPEN; + hotkey.DefHotKey = MAKEWORD( 'O', HOTKEYF_SHIFT | HOTKEYF_ALT ); + Hotkey_Register(&hotkey); + + CLISTMENUITEM mi = { sizeof(mi) }; + mi.position = 0x7FFFFFFF; + mi.flags = CMIF_TCHAR; + mi.icolibItem = icon.hIcolib; + mi.ptszName = LPGENT("Open Folder"); + mi.pszService = MS_OPENFOLDER_OPEN; + Menu_AddMainMenuItem(&mi); return 0; } diff --git a/plugins/PluginUpdater/src/DlgUpdate.cpp b/plugins/PluginUpdater/src/DlgUpdate.cpp index 30423de1e8..61332335ae 100644 --- a/plugins/PluginUpdater/src/DlgUpdate.cpp +++ b/plugins/PluginUpdater/src/DlgUpdate.cpp @@ -649,7 +649,6 @@ static int ScanFolder(const TCHAR *tszFolder, size_t cbBaseLen, int level, const } __except (EXCEPTION_EXECUTE_HANDLER) { - ZeroMemory(szMyHash, 0); // smth went wrong, reload a file from scratch } } diff --git a/plugins/Popup/src/skin.cpp b/plugins/Popup/src/skin.cpp index 1d775b8d95..fe16ce2fb4 100644 --- a/plugins/Popup/src/skin.cpp +++ b/plugins/Popup/src/skin.cpp @@ -804,7 +804,6 @@ bool PopupSkin::load(LPCTSTR dir) HANDLE hFind = FindFirstFile(_T("*.popupskin"), &ffd); while (hFind != INVALID_HANDLE_VALUE) { loadSkin(ffd.cFileName); - break; if (!FindNextFile(hFind, &ffd)) break; } diff --git a/plugins/Scriver/src/input.cpp b/plugins/Scriver/src/input.cpp index a200330596..41e2227a7a 100644 --- a/plugins/Scriver/src/input.cpp +++ b/plugins/Scriver/src/input.cpp @@ -384,7 +384,7 @@ BOOL HandleLinkClick(HINSTANCE hInstance, HWND hwndDlg, HWND hwndFocus, ENLINK * tr.lpstrText = (LPWSTR)mir_alloc(sizeof(TCHAR)*(tr.chrg.cpMax - tr.chrg.cpMin + 8)); SendMessage(lParam->nmhdr.hwndFrom, EM_GETTEXTRANGE, 0, (LPARAM)&tr); if (_tcschr(tr.lpstrText, _T('@')) != NULL && _tcschr(tr.lpstrText, _T(':')) == NULL && _tcschr(tr.lpstrText, _T('/')) == NULL) { - MoveMemory(tr.lpstrText + sizeof(TCHAR)* 7, tr.lpstrText, sizeof(TCHAR)*(tr.chrg.cpMax - tr.chrg.cpMin + 1)); + MoveMemory(tr.lpstrText + 7, tr.lpstrText, sizeof(TCHAR)*(tr.chrg.cpMax - tr.chrg.cpMin + 1)); CopyMemory(tr.lpstrText, _T("mailto:"), sizeof(TCHAR)* 7); } diff --git a/plugins/Scriver/src/msglog.cpp b/plugins/Scriver/src/msglog.cpp index 096e106ea1..e6a7d453d0 100644 --- a/plugins/Scriver/src/msglog.cpp +++ b/plugins/Scriver/src/msglog.cpp @@ -894,7 +894,6 @@ void StreamInEvents(HWND hwndDlg, HANDLE hDbEventFirst, int count, int fAppend) GETTEXTLENGTHEX gtxl = { 0 }; gtxl.flags = GTL_DEFAULT | GTL_PRECISE | GTL_NUMCHARS; gtxl.codepage = 1200; - gtxl.codepage = 1200; fi.chrg.cpMin = SendDlgItemMessage(hwndDlg, IDC_LOG, EM_GETTEXTLENGTHEX, (WPARAM)>xl, 0); sel.cpMin = sel.cpMax = GetRichTextLength(GetDlgItem(hwndDlg, IDC_LOG), dat->windowData.codePage, FALSE); SendDlgItemMessage(hwndDlg, IDC_LOG, EM_EXSETSEL, 0, (LPARAM)&sel); diff --git a/plugins/SimpleStatusMsg/src/main.cpp b/plugins/SimpleStatusMsg/src/main.cpp index cafd032c92..d514176267 100644 --- a/plugins/SimpleStatusMsg/src/main.cpp +++ b/plugins/SimpleStatusMsg/src/main.cpp @@ -1447,12 +1447,12 @@ static int ChangeStatusMsgPrebuild(WPARAM wParam, LPARAM lParam) log2file("ChangeStatusMsgPrebuild()"); #endif PROTOACCOUNT **pa; - int iStatusMenuItemCount = 0, count, i; + int iStatusMenuItemCount = 0, count; DWORD iStatusMsgFlags = 0; ProtoEnumAccounts(&count, &pa); hProtoStatusMenuItem = (HANDLE *)mir_realloc(hProtoStatusMenuItem, sizeof(HANDLE) * count); - for (i = 0; i < count; ++i) + for (int i = 0; i < count; ++i) { if (!IsAccountEnabled(pa[i])) continue; @@ -1482,11 +1482,8 @@ static int ChangeStatusMsgPrebuild(WPARAM wParam, LPARAM lParam) mi.popupPosition = 500084000; mi.position = 2000040000; - for (i = 0; i < count; ++i) + for (int i = 0; i < count; ++i) { - char szSetting[80]; - int iProtoFlags; - if (!IsAccountEnabled(pa[i])) continue; @@ -1499,18 +1496,20 @@ static int ChangeStatusMsgPrebuild(WPARAM wParam, LPARAM lParam) if (!pa[i]->bIsVisible) continue; + char szSetting[80]; mir_snprintf(szSetting, SIZEOF(szSetting), "Proto%sFlags", pa[i]->szModuleName); - iProtoFlags = db_get_b(NULL, "SimpleStatusMsg", szSetting, PROTO_DEFAULT); + int iProtoFlags = db_get_b(NULL, "SimpleStatusMsg", szSetting, PROTO_DEFAULT); if (iProtoFlags & PROTO_NO_MSG || iProtoFlags & PROTO_THIS_MSG) continue; + TCHAR szBuffer[256]; if (CallService(MS_PROTO_ISACCOUNTLOCKED,0,(LPARAM)pa[i]->szModuleName)) { - TCHAR szBuffer[256]; mir_sntprintf(szBuffer, SIZEOF(szBuffer), TranslateT("%s (locked)"), pa[i]->tszAccountName); mi.ptszPopupName = szBuffer; } - else mi.ptszPopupName = pa[i]->tszAccountName; + else + mi.ptszPopupName = pa[i]->tszAccountName; hProtoStatusMenuItem[i] = Menu_AddStatusMenuItem(&mi); } @@ -1578,7 +1577,7 @@ static int OnIdleChanged(WPARAM, LPARAM lParam) static int CSStatusChange(WPARAM wParam, LPARAM lParam) { - PROTOCOLSETTINGEX** ps = *(PROTOCOLSETTINGEX***)wParam; + PROTOCOLSETTINGEX **ps = *(PROTOCOLSETTINGEX***)wParam; int status_mode, CSProtoCount; char szSetting[80]; TCHAR *msg = NULL; @@ -1657,13 +1656,11 @@ static int CSStatusChange(WPARAM wParam, LPARAM lParam) static TCHAR *ParseWinampSong(ARGUMENTSINFO *ai) { - TCHAR *ptszWinampTitle; - if (ai->argc != 1) return NULL; ai->flags |= AIF_DONTPARSE; - ptszWinampTitle = GetWinampSong(); + TCHAR *ptszWinampTitle = GetWinampSong(); if (ptszWinampTitle != NULL) { mir_free(g_ptszWinampSong); @@ -1677,11 +1674,10 @@ static TCHAR *ParseWinampSong(ARGUMENTSINFO *ai) static TCHAR *ParseDate(ARGUMENTSINFO *ai) { - TCHAR szStr[128] = {0}; - if (ai->argc != 1) return NULL; + TCHAR szStr[128] = {0}; ai->flags |= AIF_DONTPARSE; GetDateFormat(LOCALE_USER_DEFAULT, DATE_SHORTDATE, NULL, NULL, szStr, SIZEOF(szStr)); @@ -1769,7 +1765,7 @@ static int OnAccListChanged(WPARAM wParam, LPARAM lParam) return 0; } -static int OnModulesLoaded(WPARAM wParam, LPARAM lParam) +static int OnModulesLoaded(WPARAM, LPARAM) { #ifdef _DEBUG log2file("### Session started ###"); @@ -1822,7 +1818,7 @@ static int OnModulesLoaded(WPARAM wParam, LPARAM lParam) g_ptszWinampSong = mir_tstrdup(_T("SimpleStatusMsg"));*/ if (db_get_b(NULL, "SimpleStatusMsg", "UpdateMsgOn", 1)) - g_uUpdateMsgTimer = SetTimer(NULL, 0, db_get_w(NULL, "SimpleStatusMsg", "UpdateMsgInt", 10) * 1000, (TIMERPROC)UpdateMsgTimerProc); + g_uUpdateMsgTimer = SetTimer(NULL, 0, db_get_w(NULL, "SimpleStatusMsg", "UpdateMsgInt", 10) * 1000, UpdateMsgTimerProc); if (ServiceExists(MS_CS_SETSTATUSEX)) HookEvent(ME_CS_STATUSCHANGEEX, CSStatusChange); @@ -1832,7 +1828,7 @@ static int OnModulesLoaded(WPARAM wParam, LPARAM lParam) if (!ServiceExists(MS_SS_GETPROFILECOUNT)) { if (db_get_b(NULL, "SimpleStatusMsg", "GlobalStatusDelay", 1)) - SetTimer(NULL, 0, db_get_w(NULL, "SimpleStatusMsg", "SetStatusDelay", 300), (TIMERPROC)SetStartupStatusGlobal); + SetTimer(NULL, 0, db_get_w(NULL, "SimpleStatusMsg", "SetStatusDelay", 300), SetStartupStatusGlobal); else { g_uSetStatusTimer = (UINT_PTR *)mir_alloc(sizeof(UINT_PTR) * accounts->count); for (int i = 0; i < accounts->count; ++i) { @@ -1844,7 +1840,7 @@ static int OnModulesLoaded(WPARAM wParam, LPARAM lParam) char szSetting[80]; mir_snprintf(szSetting, SIZEOF(szSetting), "Set%sStatusDelay", accounts->pa[i]->szModuleName); - g_uSetStatusTimer[i] = SetTimer(NULL, 0, db_get_w(NULL, "SimpleStatusMsg", szSetting, 300), (TIMERPROC)SetStartupStatusProc); + g_uSetStatusTimer[i] = SetTimer(NULL, 0, db_get_w(NULL, "SimpleStatusMsg", szSetting, 300), SetStartupStatusProc); } } } @@ -1852,7 +1848,7 @@ static int OnModulesLoaded(WPARAM wParam, LPARAM lParam) return 0; } -static int OnOkToExit(WPARAM wParam, LPARAM lParam) +static int OnOkToExit(WPARAM, LPARAM) { if (accounts->statusCount) { char szSetting[80]; @@ -1875,7 +1871,7 @@ static int OnOkToExit(WPARAM wParam, LPARAM lParam) return 0; } -static int OnPreShutdown(WPARAM wParam, LPARAM lParam) +static int OnPreShutdown(WPARAM, LPARAM) { if (!accounts->statusMsgFlags) return 0; diff --git a/plugins/SimpleStatusMsg/src/msgbox.cpp b/plugins/SimpleStatusMsg/src/msgbox.cpp index d71d7aa763..dd2c06f3b1 100644 --- a/plugins/SimpleStatusMsg/src/msgbox.cpp +++ b/plugins/SimpleStatusMsg/src/msgbox.cpp @@ -244,7 +244,7 @@ HWND WINAPI CreateRecentComboBoxEx(HWND hwndDlg, struct MsgBoxData *data) mir_snprintf(buff, SIZEOF(buff), "SMsg%d", j); j--; if (!db_get_ts(NULL, "SimpleStatusMsg", buff, &dbv)) { - if (dbv.ptszVal != NULL && dbv.ptszVal != '\0') { + if (dbv.ptszVal != NULL && *dbv.ptszVal != '\0') { found = TRUE; cbei.iItem = -1; cbei.pszText = (LPTSTR)dbv.ptszVal; diff --git a/plugins/Spamotron/src/utils.cpp b/plugins/Spamotron/src/utils.cpp index ef78a9f29d..ce8730a7b9 100644 --- a/plugins/Spamotron/src/utils.cpp +++ b/plugins/Spamotron/src/utils.cpp @@ -206,25 +206,24 @@ int get_response_num(const TCHAR *str) return i; } -TCHAR* get_response(TCHAR* dst, unsigned int dstlen, int num) +TCHAR* get_response(TCHAR *dst, unsigned int dstlen, int num) { - int i = 0; - TCHAR *tmp, *src = NULL; if (num < 0) return dst; - src = (TCHAR*)malloc(MAX_BUFFER_LENGTH * sizeof(TCHAR)); + TCHAR *src = (TCHAR*)malloc(MAX_BUFFER_LENGTH * sizeof(TCHAR)); + int i = 0; if (src != NULL) { _getOptS(src, MAX_BUFFER_LENGTH, "Response", defaultResponse); - _tcscpy(src, src); - tmp = _tcstok(src, L"\r\n"); + _tcsncpy(dst, src, dstlen); + TCHAR *tmp = _tcstok(src, L"\r\n"); while (tmp) { if (i == num) { - _tcscpy(dst, tmp); + _tcsncpy(dst, tmp, dstlen); free(src); return dst; } i++; - tmp = _tcstok(NULL, L"\r\n"); + tmp = _tcstok(src, L"\r\n"); } free(src); } diff --git a/plugins/TabSRMM/src/buttonsbar.cpp b/plugins/TabSRMM/src/buttonsbar.cpp index 3d46372ed1..2f21dfed4e 100644 --- a/plugins/TabSRMM/src/buttonsbar.cpp +++ b/plugins/TabSRMM/src/buttonsbar.cpp @@ -414,6 +414,11 @@ void BB_UpdateIcons(HWND hdlg, TWindowData *dat) void TSAPI BB_InitDlgButtons(TWindowData *dat) { + if (dat == 0) + return; + HWND hdlg = dat->hwnd; + if (hdlg == 0) + return; RECT rect; int i; int lwidth = 0, rwidth = 0; @@ -425,9 +430,7 @@ void TSAPI BB_InitDlgButtons(TWindowData *dat) int cx = 0, cy = 0; int lcount = LButtonsList.getCount(); int rcount = RButtonsList.getCount(); - HWND hdlg = dat->hwnd; - if (dat == 0 || hdlg == 0) { return; } if (CSkin::m_skinEnabled && !SkinItems[ID_EXTBKBUTTONSNPRESSED].IGNORED && !SkinItems[ID_EXTBKBUTTONSPRESSED].IGNORED && !SkinItems[ID_EXTBKBUTTONSMOUSEOVER].IGNORED) { isThemed = FALSE; diff --git a/plugins/TabSRMM/src/container.cpp b/plugins/TabSRMM/src/container.cpp index 1962c1b8e1..5f74bbd09c 100644 --- a/plugins/TabSRMM/src/container.cpp +++ b/plugins/TabSRMM/src/container.cpp @@ -619,7 +619,7 @@ static INT_PTR CALLBACK DlgProcContainer(HWND hwndDlg, UINT msg, WPARAM wParam, LONG y_pad = M.GetByte("y-pad", 3) + ((pContainer->dwFlags & CNT_TABSBOTTOM) ? 1 : 0); if (pContainer->dwFlagsEx & TCF_FLAT) - y_pad += 1; //(pContainer->dwFlags & CNT_TABSBOTTOM ? 1 : 2); + y_pad++; //(pContainer->dwFlags & CNT_TABSBOTTOM ? 1 : 2); TabCtrl_SetPadding(GetDlgItem(hwndDlg, IDC_MSGTABS), x_pad, y_pad); @@ -838,14 +838,12 @@ panel_found: } else if (((LPNMHDR)lParam)->code == NM_RCLICK) { POINT pt; - MCONTACT hContact = 0; - HMENU hMenu; - GetCursorPos(&pt); + MCONTACT hContact = 0; SendMessage(pContainer->hwndActive, DM_QUERYHCONTACT, 0, (LPARAM)&hContact); if (hContact) { int iSel = 0; - hMenu = (HMENU) CallService(MS_CLIST_MENUBUILDCONTACT, hContact, 0); + HMENU hMenu = (HMENU) CallService(MS_CLIST_MENUBUILDCONTACT, hContact, 0); iSel = TrackPopupMenu(hMenu, TPM_RETURNCMD, pt.x, pt.y, 0, hwndDlg, NULL); if (iSel) CallService(MS_CLIST_MENUPROCESSCOMMAND, MAKEWPARAM(LOWORD(iSel), MPCF_CONTACTMENU), hContact); @@ -1000,9 +998,7 @@ panel_found: { RECT rc; LONG dwNewLeft; - BOOL skinnedMode = bSkinned; - - skinnedMode |= (IsThemeActive() ? 1 : 0); + BOOL skinnedMode = bSkinned | (IsThemeActive() ? 1 : 0); GetWindowRect(hwndDlg, &rc); diff --git a/plugins/TabSRMM/src/controls.cpp b/plugins/TabSRMM/src/controls.cpp index f26f1e3990..b09bfdc44b 100644 --- a/plugins/TabSRMM/src/controls.cpp +++ b/plugins/TabSRMM/src/controls.cpp @@ -448,7 +448,7 @@ void CMenuBar::updateState(const HMENU hMenu) const if (dat) { ::CheckMenuItem(hMenu, ID_VIEW_SHOWMENUBAR, MF_BYCOMMAND | m_pContainer->dwFlags & CNT_NOMENUBAR ? MF_UNCHECKED : MF_CHECKED); ::CheckMenuItem(hMenu, ID_VIEW_SHOWSTATUSBAR, MF_BYCOMMAND | m_pContainer->dwFlags & CNT_NOSTATUSBAR ? MF_UNCHECKED : MF_CHECKED); - ::CheckMenuItem(hMenu, ID_VIEW_SHOWAVATAR, MF_BYCOMMAND | dat->bShowAvatar ? MF_CHECKED : MF_UNCHECKED); + ::CheckMenuItem(hMenu, ID_VIEW_SHOWAVATAR, MF_BYCOMMAND | (dat->bShowAvatar ? MF_CHECKED : MF_UNCHECKED)); ::CheckMenuItem(hMenu, ID_VIEW_SHOWTITLEBAR, MF_BYCOMMAND | m_pContainer->dwFlags & CNT_NOTITLE ? MF_UNCHECKED : MF_CHECKED); ::EnableMenuItem(hMenu, ID_VIEW_SHOWTITLEBAR, CSkin::m_skinEnabled && CSkin::m_frameSkins ? MF_GRAYED : MF_ENABLED); diff --git a/plugins/TabSRMM/src/hotkeyhandler.cpp b/plugins/TabSRMM/src/hotkeyhandler.cpp index c5fd0dc581..adeaedb97e 100644 --- a/plugins/TabSRMM/src/hotkeyhandler.cpp +++ b/plugins/TabSRMM/src/hotkeyhandler.cpp @@ -520,7 +520,7 @@ LONG_PTR CALLBACK HotkeyHandlerDlgProc(HWND hwndDlg, UINT msg, WPARAM wParam, LP ack.type = ACKTYPE_MESSAGE; ack.result = ACKRESULT_SUCCESS; - if (job->hContact && job->iAcksNeeded && job->hContact && job->iStatus == SendQueue::SQ_INPROGRESS) { + if (job->hContact && job->iAcksNeeded && job->iStatus == SendQueue::SQ_INPROGRESS) { if (IsWindow(job->hOwnerWnd)) ::SendMessage(job->hOwnerWnd, HM_EVENTSENT, (WPARAM)MAKELONG(wParam, 0), (LPARAM)&ack); else diff --git a/plugins/TabSRMM/src/msgdialog.cpp b/plugins/TabSRMM/src/msgdialog.cpp index ab677d6451..9b334fb22f 100644 --- a/plugins/TabSRMM/src/msgdialog.cpp +++ b/plugins/TabSRMM/src/msgdialog.cpp @@ -3231,7 +3231,7 @@ quote_from_last: } _dlgReturn(hwndDlg, TRUE); } - { + if(dat) { TContainerData *pContainer = dat->pContainer; if (dat->iOpenJobs > 0 && lParam != 2) { if (dat->dwFlags & MWF_ERRORSTATE) { diff --git a/plugins/TabSRMM/src/msglog.cpp b/plugins/TabSRMM/src/msglog.cpp index a448be6be0..e5299a53da 100644 --- a/plugins/TabSRMM/src/msglog.cpp +++ b/plugins/TabSRMM/src/msglog.cpp @@ -126,13 +126,12 @@ static void TrimMessage(TCHAR *msg) void TSAPI CacheLogFonts() { - int i; HDC hdc = GetDC(NULL); logPixelSY = GetDeviceCaps(hdc, LOGPIXELSY); ReleaseDC(NULL, hdc); - ZeroMemory(logfonts, sizeof(LOGFONTA) * MSGDLGFONTCOUNT + 2); - for (i = 0; i < MSGDLGFONTCOUNT; i++) { + ZeroMemory(logfonts, sizeof(LOGFONTA) * (MSGDLGFONTCOUNT + 2)); + for (int i = 0; i < MSGDLGFONTCOUNT; i++) { LoadLogfont(i, &logfonts[i], &fontcolors[i], FONTMODULE); mir_snprintf(rtfFontsGlobal[i], SIZEOF(rtfFontsGlobal[i]), "\\f%u\\cf%u\\b%d\\i%d\\ul%d\\fs%u", i, i, logfonts[i].lfWeight >= FW_BOLD ? 1 : 0, logfonts[i].lfItalic, logfonts[i].lfUnderline, 2 * abs(logfonts[i].lfHeight) * 74 / logPixelSY); } @@ -149,7 +148,7 @@ void TSAPI CacheLogFonts() COLORREF clr; LOGFONTA lf; - for (i = 0; i < IPFONTCOUNT; i++) { + for (int i = 0; i < IPFONTCOUNT; i++) { if (CInfoPanel::m_ipConfig.hFonts[i]) DeleteObject(CInfoPanel::m_ipConfig.hFonts[i]); LoadLogfont(i + 100, &lf, &clr, FONTMODULE); diff --git a/plugins/TabSRMM/src/sendqueue.cpp b/plugins/TabSRMM/src/sendqueue.cpp index fd5b0f1342..d9a47724af 100644 --- a/plugins/TabSRMM/src/sendqueue.cpp +++ b/plugins/TabSRMM/src/sendqueue.cpp @@ -580,7 +580,7 @@ void SendQueue::recallFailed(const TWindowData *dat, int iEntry) const void SendQueue::UpdateSaveAndSendButton(TWindowData *dat) { if (dat) { - HWND hwndDlg = dat->hwnd; + HWND hwndDlg = dat->hwnd; GETTEXTLENGTHEX gtxl = {0}; gtxl.codepage = CP_UTF8; @@ -829,7 +829,7 @@ int SendQueue::doSendLater(int iJobIndex, TWindowData *dat, MCONTACT hContact, b } if (iJobIndex >= 0 && iJobIndex < NR_SENDJOBS) { - SendJob* job = &m_jobs[iJobIndex]; + SendJob *job = &m_jobs[iJobIndex]; char szKeyName[20]; TCHAR tszHeader[150]; diff --git a/plugins/TabSRMM/src/templates.cpp b/plugins/TabSRMM/src/templates.cpp index 4bd8f52bae..d03e5c81f8 100644 --- a/plugins/TabSRMM/src/templates.cpp +++ b/plugins/TabSRMM/src/templates.cpp @@ -118,11 +118,10 @@ void LoadDefaultTemplates() INT_PTR CALLBACK DlgProcTemplateEditor(HWND hwndDlg, UINT msg, WPARAM wParam, LPARAM lParam) { - TWindowData *dat; TemplateEditorInfo *teInfo = 0; TTemplateSet *tSet; int i; - dat = (TWindowData*)GetWindowLongPtr(hwndDlg, GWLP_USERDATA); + TWindowData *dat = (TWindowData*)GetWindowLongPtr(hwndDlg, GWLP_USERDATA); /* * since this dialog needs a MessageWindowData * but has no container, we can store * the extended info struct in pContainer *) diff --git a/plugins/TabSRMM/src/themes.cpp b/plugins/TabSRMM/src/themes.cpp index 44284695f3..7e2e4833ae 100644 --- a/plugins/TabSRMM/src/themes.cpp +++ b/plugins/TabSRMM/src/themes.cpp @@ -841,7 +841,6 @@ TCHAR* CImageItem::Read(const TCHAR *szFilename) m_dwFlags |= IMAGE_FLAG_DIVIDED; m_bf.BlendFlags = 0; m_bf.BlendOp = AC_SRC_OVER; - m_bf.AlphaFormat = 0; m_dwFlags |= IMAGE_PERPIXEL_ALPHA; m_bf.AlphaFormat = AC_SRC_ALPHA; if (m_inner_height <= 0 || m_inner_width <= 0) { @@ -1371,7 +1370,6 @@ void CSkin::Load(void) if (!m_tszFileName[0] || !::PathFileExists(m_tszFileName)) return; - TCHAR *p; TCHAR *szSections = (TCHAR*)mir_alloc(6004); int i = 1, j = 0; UINT data; @@ -1407,10 +1405,9 @@ void CSkin::Load(void) m_DisableScrollbars = M.GetByte("disableVScroll", 0) ? true : false; ZeroMemory(szSections, 6000); - p = szSections; GetPrivateProfileSectionNames(szSections, 3000, m_tszFileName); szSections[3001] = szSections[3000] = 0; - p = szSections; + TCHAR *p = szSections; while (lstrlen(p) > 1) { if (p[0] != '%') { p += (lstrlen(p) + 1); @@ -1719,19 +1716,19 @@ void CSkin::setupAeroSkins() float fmax = max(max(fr, fg), fb); if (fmax == fr) { - fr *= (alphafactor / 100 * 2.2); + fr *= (alphafactor / 100.0 * 2.2); fr = min(fr, 255); fb = min(fb * alphafactor / 100, 255); fg = min(fg * alphafactor / 100, 255); } else if (fmax == fg) { - fg *= (alphafactor / 100 * 2.2); + fg *= (alphafactor / 100.0 * 2.2); fg = min(fg, 255); fr = min(fr * alphafactor / 100, 255); fb = min(fb * alphafactor / 100, 255); } else { - fb *= (alphafactor / 100 * 2.2); + fb *= (alphafactor / 100.0 * 2.2); fb = min(fb, 255); fr = min(fr * alphafactor / 100, 255); fg = min(fg * alphafactor / 100, 255); @@ -1838,11 +1835,9 @@ void CSkin::setupAeroSkins() void CSkin::SkinCalcFrameWidth() { - int xBorder, yBorder, yCaption; - - xBorder = GetSystemMetrics(SM_CXSIZEFRAME); - yBorder = GetSystemMetrics(SM_CYSIZEFRAME); - yCaption = GetSystemMetrics(SM_CYCAPTION); + int xBorder = GetSystemMetrics(SM_CXSIZEFRAME); + int yBorder = GetSystemMetrics(SM_CYSIZEFRAME); + int yCaption = GetSystemMetrics(SM_CYCAPTION); m_realSkinnedFrame_left = m_SkinnedFrame_left - xBorder; m_realSkinnedFrame_right = m_SkinnedFrame_right - xBorder; diff --git a/plugins/UserInfoEx/src/Flags/svc_flagsicons.cpp b/plugins/UserInfoEx/src/Flags/svc_flagsicons.cpp index bf0329ece7..6255e8cbad 100644 --- a/plugins/UserInfoEx/src/Flags/svc_flagsicons.cpp +++ b/plugins/UserInfoEx/src/Flags/svc_flagsicons.cpp @@ -261,7 +261,6 @@ static INT_PTR ServiceCreateMergedFlagIcon(WPARAM wParam,LPARAM lParam) hrgn=CreatePolygonRgn(aptTriangle,SIZEOF(aptTriangle),WINDING); if (hrgn!=NULL) { SelectClipRgn(hdc,hrgn); - DeleteObject(hrgn); hbmPrev=(HBITMAP)SelectObject(hdc,icoi.hbmColor); if (hbmPrev!=NULL) { /* error on select? */ if (DrawIconEx(hdc,0,0,hUpperIcon,bm.bmWidth,bm.bmHeight,0,NULL,DI_NOMIRROR|DI_IMAGE)) { diff --git a/plugins/Watrack_MPD/src/main.cpp b/plugins/Watrack_MPD/src/main.cpp index 01496d9f06..171bcc853c 100755 --- a/plugins/Watrack_MPD/src/main.cpp +++ b/plugins/Watrack_MPD/src/main.cpp @@ -413,17 +413,17 @@ void RegisterPlayer() { PLAYERCELL player = {0}; player.Desc = "Music Player Daemon"; + player.Notes = _T("mpd is a nice music player for *nix which have not any gui, just daemon.\nuses very small amount of ram, cpu."); + player.URL = "http://www.musicpd.org"; player.Check = CheckPlayer; player.Init = Init; player.DeInit = DeInit; - player.GetStatus = GetStatus; player.Command = SendCommand; - player.flags = (WAT_OPT_HASURL|WAT_OPT_SINGLEINST|WAT_OPT_PLAYERINFO); + player.GetStatus = GetStatus; player.GetName = GetFileName; player.GetInfo = GetPlayerInfo; + player.flags = (WAT_OPT_HASURL|WAT_OPT_SINGLEINST|WAT_OPT_PLAYERINFO); // player.Icon = //TODO:implement icon support - player.Notes = _T("mpd is a nice music player for *nix which have not any gui, just daemon.\nuses very small amount of ram, cpu."); - player.URL = "http://www.musicpd.org"; CallService(MS_WAT_PLAYER, WAT_ACT_REGISTER, (LPARAM)&player); } } \ No newline at end of file diff --git a/plugins/Weather/src/weather_ini.cpp b/plugins/Weather/src/weather_ini.cpp index 59c618382b..066861beed 100644 --- a/plugins/Weather/src/weather_ini.cpp +++ b/plugins/Weather/src/weather_ini.cpp @@ -247,7 +247,7 @@ void LoadStationData(TCHAR *pszFile, TCHAR *pszShortFile, WIDATA *Data) }; // clean up old stuff - ZeroMemory(Data, sizeof(Data)); + ZeroMemory(Data, sizeof(*Data)); Data->Enabled = FALSE; // open the ini file diff --git a/plugins/WebView/src/webview_alerts.cpp b/plugins/WebView/src/webview_alerts.cpp index d2aeef1329..35378816ba 100644 --- a/plugins/WebView/src/webview_alerts.cpp +++ b/plugins/WebView/src/webview_alerts.cpp @@ -132,15 +132,15 @@ int PopupAlert(WPARAM wParam, LPARAM lParam) if ( db_get_b(NULL, MODULENAME, POP_USECUSTCLRS_KEY, 0)) { ppd.colorBack = db_get_dw(NULL, MODULENAME, POP_BG_CLR_KEY, Def_color_bg); - ppd.colorBack = db_get_dw(NULL, MODULENAME, POP_TXT_CLR_KEY, Def_color_txt); + ppd.colorText = db_get_dw(NULL, MODULENAME, POP_TXT_CLR_KEY, Def_color_txt); } - if ( db_get_b(NULL, MODULENAME, POP_USEWINCLRS_KEY, 0)) { + else if ( db_get_b(NULL, MODULENAME, POP_USEWINCLRS_KEY, 0)) { ppd.colorBack = GetSysColor(COLOR_BTNFACE); - ppd.colorBack = GetSysColor(COLOR_WINDOWTEXT); + ppd.colorText = GetSysColor(COLOR_WINDOWTEXT); } - if ( db_get_b(NULL, MODULENAME, POP_USESAMECLRS_KEY, 1)) { + else if ( db_get_b(NULL, MODULENAME, POP_USESAMECLRS_KEY, 1)) { ppd.colorBack = BackgoundClr; - ppd.colorBack = TextClr; + ppd.colorText = TextClr; } ppd.PluginWindowProc = NULL; diff --git a/plugins/WinterSpeak/src/DialogConfigEngine.cpp b/plugins/WinterSpeak/src/DialogConfigEngine.cpp index 2375dedcc7..9c5be8dfe9 100644 --- a/plugins/WinterSpeak/src/DialogConfigEngine.cpp +++ b/plugins/WinterSpeak/src/DialogConfigEngine.cpp @@ -109,7 +109,7 @@ void DialogConfigEngine::load(HWND window) std::vector engines = si.getAvailableEngines(); for (unsigned int i = 0; i < engines.size(); ++i) { - SendDlgItemMessage(window, IDC_SELECT_ENGINE, CB_ADDSTRING, 0, (long)(engines[i].c_str())); + SendDlgItemMessage(window, IDC_SELECT_ENGINE, CB_ADDSTRING, 0, (LPARAM)(engines[i].c_str())); } VoiceDesc desc = m_db.getVoiceDesc(); @@ -120,7 +120,7 @@ void DialogConfigEngine::load(HWND window) SendDlgItemMessage(window, IDC_SLIDER_RATE, TBM_SETPOS, TRUE, desc.rate); // select the speech engine - SendDlgItemMessage(window, IDC_SELECT_ENGINE, CB_SELECTSTRING, 0, reinterpret_cast(desc.engine.c_str())); + SendDlgItemMessage(window, IDC_SELECT_ENGINE, CB_SELECTSTRING, 0, reinterpret_cast(desc.engine.c_str())); // initialise the welcome message box SetDlgItemText(window, IDC_WELCOME_MSG, m_db.getWelcomeMessage().c_str()); @@ -158,13 +158,13 @@ void DialogConfigEngine::updateVoices(HWND window) for (unsigned int i = 0; i < voices.size(); ++i) { SendDlgItemMessage(window, IDC_SELECT_VOICE, CB_ADDSTRING, 0, - (long)voices[i].c_str()); + (LPARAM)voices[i].c_str()); } // get the voice saved in the database std::wstring voice = m_db.getVoiceDesc().voice; - if (FAILED(SendDlgItemMessage(window, IDC_SELECT_VOICE, CB_FINDSTRINGEXACT, 0, (long)voice.c_str()))) + if (FAILED(SendDlgItemMessage(window, IDC_SELECT_VOICE, CB_FINDSTRINGEXACT, 0, (LPARAM)voice.c_str()))) { // select the first one SendDlgItemMessage(window, IDC_SELECT_VOICE, CB_SETCURSEL , 0, 0); diff --git a/plugins/YARelay/src/main.cpp b/plugins/YARelay/src/main.cpp index e41a97859c..eac2d93b3f 100644 --- a/plugins/YARelay/src/main.cpp +++ b/plugins/YARelay/src/main.cpp @@ -156,7 +156,7 @@ static int MessageEventAdded(WPARAM wParam, LPARAM lParam) // get sender's uin CONTACTINFO ci = { sizeof(ci) }; ci.dwFlag = CNF_UNIQUEID; - if (CallService(MS_CONTACT_GETCONTACTINFO, 0, (long)&ci) == 0){ + if (CallService(MS_CONTACT_GETCONTACTINFO, 0, (LPARAM)&ci) == 0){ if (ci.type == CNFT_ASCIIZ) _tcsncpy_s(buf, ci.pszVal, _TRUNCATE); else if (ci.type == CNFT_BYTE) -- cgit v1.2.3