diff --git a/PowerEditor/src/Notepad_plus.cpp b/PowerEditor/src/Notepad_plus.cpp index 0b6b5a86..3d653cb9 100644 --- a/PowerEditor/src/Notepad_plus.cpp +++ b/PowerEditor/src/Notepad_plus.cpp @@ -2824,7 +2824,7 @@ void Notepad_plus::setTitle() if (_isAdministrator) result += TEXT(" [Administrator]"); - ::SendMessage(_pPublicInterface->getHSelf(), WM_SETTEXT, 0, (LPARAM)result.c_str()); + ::SendMessage(_pPublicInterface->getHSelf(), WM_SETTEXT, 0, reinterpret_cast(result.c_str())); } void Notepad_plus::activateNextDoc(bool direction) diff --git a/PowerEditor/src/Parameters.cpp b/PowerEditor/src/Parameters.cpp index 1ca191e7..d880cf12 100644 --- a/PowerEditor/src/Parameters.cpp +++ b/PowerEditor/src/Parameters.cpp @@ -725,14 +725,11 @@ generic_string ThemeSwitcher::getThemeFromXmlFileName(const TCHAR *xmlFullPath) if (!xmlFullPath || !xmlFullPath[0]) return generic_string(); generic_string fn(::PathFindFileName(xmlFullPath)); - PathRemoveExtension((TCHAR *)fn.c_str()); + PathRemoveExtension(const_cast(fn.c_str())); return fn; } -#pragma warning(disable : 4996) - - winVer getWindowsVersion() { OSVERSIONINFOEX osvi; diff --git a/PowerEditor/src/ScitillaComponent/AutoCompletion.cpp b/PowerEditor/src/ScitillaComponent/AutoCompletion.cpp index d55742ac..c0ff618a 100644 --- a/PowerEditor/src/ScitillaComponent/AutoCompletion.cpp +++ b/PowerEditor/src/ScitillaComponent/AutoCompletion.cpp @@ -539,7 +539,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m { char userMatchedChar[2] = { '\0', '\0' }; userMatchedChar[0] = matchedPairs[i].second; - _pEditView->execute(SCI_INSERTTEXT, caretPos, (LPARAM)userMatchedChar); + _pEditView->execute(SCI_INSERTTEXT, caretPos, reinterpret_cast(userMatchedChar)); return; } } @@ -559,7 +559,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m { matchedChars = ")"; - _insertedMatchedChars.add(MatchedCharInserted(char(character), caretPos - 1)); + _insertedMatchedChars.add(MatchedCharInserted(static_cast(character), caretPos - 1)); } } break; @@ -570,7 +570,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m if (isCharNextBlank || isInSandwich) { matchedChars = "]"; - _insertedMatchedChars.add(MatchedCharInserted(char(character), caretPos - 1)); + _insertedMatchedChars.add(MatchedCharInserted(static_cast(character), caretPos - 1)); } } break; @@ -581,7 +581,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m if (isCharNextBlank || isInSandwich) { matchedChars = "}"; - _insertedMatchedChars.add(MatchedCharInserted(char(character), caretPos - 1)); + _insertedMatchedChars.add(MatchedCharInserted(static_cast(character), caretPos - 1)); } } break; @@ -591,7 +591,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m { if (!_insertedMatchedChars.isEmpty()) { - int pos = _insertedMatchedChars.search('"', char(character), caretPos); + int pos = _insertedMatchedChars.search('"', static_cast(character), caretPos); if (pos != -1) { _pEditView->execute(SCI_DELETERANGE, pos, 1); @@ -606,7 +606,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m (charPrev == '{' && isCharNextBlank) || (isCharPrevBlank && charNext == '}')) { matchedChars = "\""; - _insertedMatchedChars.add(MatchedCharInserted(char(character), caretPos - 1)); + _insertedMatchedChars.add(MatchedCharInserted(static_cast(character), caretPos - 1)); } } break; @@ -615,7 +615,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m { if (!_insertedMatchedChars.isEmpty()) { - int pos = _insertedMatchedChars.search('\'', char(character), caretPos); + int pos = _insertedMatchedChars.search('\'', static_cast(character), caretPos); if (pos != -1) { _pEditView->execute(SCI_DELETERANGE, pos, 1); @@ -630,7 +630,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m (charPrev == '{' && isCharNextBlank) || (isCharPrevBlank && charNext == '}')) { matchedChars = "'"; - _insertedMatchedChars.add(MatchedCharInserted(char(character), caretPos - 1)); + _insertedMatchedChars.add(MatchedCharInserted(static_cast(character), caretPos - 1)); } } break; @@ -671,7 +671,7 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m startChar = '{'; } - int pos = _insertedMatchedChars.search(startChar, char(character), caretPos); + int pos = _insertedMatchedChars.search(startChar, static_cast(character), caretPos); if (pos != -1) { _pEditView->execute(SCI_DELETERANGE, pos, 1); @@ -683,11 +683,11 @@ void AutoCompletion::insertMatchedChars(int character, const MatchedPairConf & m default: if (!_insertedMatchedChars.isEmpty()) - _insertedMatchedChars.removeInvalidElements(MatchedCharInserted(char(character), caretPos - 1)); + _insertedMatchedChars.removeInvalidElements(MatchedCharInserted(static_cast(character), caretPos - 1)); } if (matchedChars) - _pEditView->execute(SCI_INSERTTEXT, caretPos, (LPARAM)matchedChars); + _pEditView->execute(SCI_INSERTTEXT, caretPos, reinterpret_cast(matchedChars)); } diff --git a/PowerEditor/src/ScitillaComponent/Buffer.cpp b/PowerEditor/src/ScitillaComponent/Buffer.cpp index c6354f19..e8c92e30 100644 --- a/PowerEditor/src/ScitillaComponent/Buffer.cpp +++ b/PowerEditor/src/ScitillaComponent/Buffer.cpp @@ -1408,7 +1408,7 @@ bool FileManager::loadFileData(Document doc, const TCHAR * filename, char* data, TCHAR * name = NppParameters::getInstance()->getELCFromIndex(id)._name; WcharMbcsConvertor *wmc = WcharMbcsConvertor::getInstance(); const char *pName = wmc->wchar2char(name, CP_ACP); - _pscratchTilla->execute(SCI_SETLEXERLANGUAGE, 0, (LPARAM)pName); + _pscratchTilla->execute(SCI_SETLEXERLANGUAGE, 0, reinterpret_cast(pName)); } if (encoding != -1) @@ -1462,14 +1462,14 @@ bool FileManager::loadFileData(Document doc, const TCHAR * filename, char* data, if (encoding == SC_CP_UTF8) { // Pass through UTF-8 (this does not check validity of characters, thus inserting a multi-byte character in two halfs is working) - _pscratchTilla->execute(SCI_APPENDTEXT, lenFile, (LPARAM)data); + _pscratchTilla->execute(SCI_APPENDTEXT, lenFile, reinterpret_cast(data)); } else { WcharMbcsConvertor* wmc = WcharMbcsConvertor::getInstance(); int newDataLen = 0; const char *newData = wmc->encode(encoding, SC_CP_UTF8, data, static_cast(lenFile), &newDataLen, &incompleteMultibyteChar); - _pscratchTilla->execute(SCI_APPENDTEXT, newDataLen, (LPARAM)newData); + _pscratchTilla->execute(SCI_APPENDTEXT, newDataLen, reinterpret_cast(newData)); } if (format == EolType::unknown) @@ -1478,7 +1478,7 @@ bool FileManager::loadFileData(Document doc, const TCHAR * filename, char* data, else { lenConvert = unicodeConvertor->convert(data, lenFile); - _pscratchTilla->execute(SCI_APPENDTEXT, lenConvert, (LPARAM)(unicodeConvertor->getNewBuf())); + _pscratchTilla->execute(SCI_APPENDTEXT, lenConvert, reinterpret_cast(unicodeConvertor->getNewBuf())); if (format == EolType::unknown) format = getEOLFormatForm(unicodeConvertor->getNewBuf(), unicodeConvertor->getNewSize(), EolType::unknown); } diff --git a/PowerEditor/src/ScitillaComponent/FindReplaceDlg.cpp b/PowerEditor/src/ScitillaComponent/FindReplaceDlg.cpp index 4d68dffc..069c9536 100644 --- a/PowerEditor/src/ScitillaComponent/FindReplaceDlg.cpp +++ b/PowerEditor/src/ScitillaComponent/FindReplaceDlg.cpp @@ -45,20 +45,20 @@ void addText2Combo(const TCHAR * txt2add, HWND hCombo) if (!hCombo) return; if (!lstrcmp(txt2add, TEXT(""))) return; - auto i = ::SendMessage(hCombo, CB_FINDSTRINGEXACT, (WPARAM)-1, (LPARAM)txt2add); + auto i = ::SendMessage(hCombo, CB_FINDSTRINGEXACT, static_cast(-1), reinterpret_cast(txt2add)); if (i != CB_ERR) // found { ::SendMessage(hCombo, CB_DELETESTRING, i, 0); } - i = ::SendMessage(hCombo, CB_INSERTSTRING, 0, (LPARAM)txt2add); + i = ::SendMessage(hCombo, CB_INSERTSTRING, 0, reinterpret_cast(txt2add)); ::SendMessage(hCombo, CB_SETCURSEL, i, 0); }; generic_string getTextFromCombo(HWND hCombo) { TCHAR str[FINDREPLACE_MAXLENGTH]; - ::SendMessage(hCombo, WM_GETTEXT, FINDREPLACE_MAXLENGTH - 1, (LPARAM)str); + ::SendMessage(hCombo, WM_GETTEXT, FINDREPLACE_MAXLENGTH - 1, reinterpret_cast(str)); return generic_string(str); }; @@ -464,7 +464,7 @@ void Finder::gotoFoundLine() const FoundInfo fInfo = *(_pMainFoundInfos->begin() + lno); // Switch to another document - ::SendMessage(::GetParent(_hParent), WM_DOOPEN, 0, (LPARAM)fInfo._fullPath.c_str()); + ::SendMessage(::GetParent(_hParent), WM_DOOPEN, 0, reinterpret_cast(fInfo._fullPath.c_str())); Searching::displaySectionCentered(fInfo._start, fInfo._end, *_ppEditView); // Then we colourise the double clicked line @@ -1984,7 +1984,7 @@ void FindReplaceDlg::findAllIn(InWhat op) _pFinder->_scintView.display(); _pFinder->display(); - ::SendMessage(_hParent, NPPM_DMMHIDE, 0, (LPARAM)_pFinder->getHSelf()); + ::SendMessage(_hParent, NPPM_DMMHIDE, 0, reinterpret_cast(_pFinder->getHSelf())); ::UpdateWindow(_hParent); justCreated = true; } @@ -1995,7 +1995,7 @@ void FindReplaceDlg::findAllIn(InWhat op) // Send the address of _MarkingsStruct to the lexer char ptrword[sizeof(void*)*2+1]; sprintf(ptrword, "%p", &_pFinder->_markingsStruct); - _pFinder->_scintView.execute(SCI_SETPROPERTY, (WPARAM)"@MarkingsStruct", (LPARAM)ptrword); + _pFinder->_scintView.execute(SCI_SETPROPERTY, reinterpret_cast("@MarkingsStruct"), reinterpret_cast(ptrword)); } ::SendMessage(_pFinder->getHSelf(), WM_SIZE, 0, 0); @@ -2023,12 +2023,12 @@ void FindReplaceDlg::findAllIn(InWhat op) else { // Show finder - ::SendMessage(_hParent, NPPM_DMMSHOW, 0, (LPARAM)_pFinder->getHSelf()); + ::SendMessage(_hParent, NPPM_DMMSHOW, 0, reinterpret_cast(_pFinder->getHSelf())); getFocus(); // no hits } } else // error - search folder doesn't exist - ::SendMessage(_hSelf, WM_NEXTDLGCTL, (WPARAM)::GetDlgItem(_hSelf, IDD_FINDINFILES_DIR_COMBO), TRUE); + ::SendMessage(_hSelf, WM_NEXTDLGCTL, reinterpret_cast(::GetDlgItem(_hSelf, IDD_FINDINFILES_DIR_COMBO)), TRUE); } Finder * FindReplaceDlg::createFinder() @@ -2039,7 +2039,7 @@ Finder * FindReplaceDlg::createFinder() tTbData data = { 0 }; pFinder->create(&data, false); - ::SendMessage(_hParent, NPPM_MODELESSDIALOG, MODELESSDIALOGREMOVE, (WPARAM)pFinder->getHSelf()); + ::SendMessage(_hParent, NPPM_MODELESSDIALOG, MODELESSDIALOGREMOVE, reinterpret_cast(pFinder->getHSelf())); // define the default docking behaviour data.uMask = DWS_DF_CONT_BOTTOM | DWS_ICONTAB | DWS_ADDINFO; data.hIconTab = (HICON)::LoadImage(_hInst, MAKEINTRESOURCE(IDI_FIND_RESULT_ICON), IMAGE_ICON, 0, 0, LR_LOADMAP3DCOLORS | LR_LOADTRANSPARENT); @@ -2050,12 +2050,12 @@ Finder * FindReplaceDlg::createFinder() // the dlgDlg should be the index of funcItem where the current function pointer is // in this case is DOCKABLE_DEMO_INDEX data.dlgID = 0; - ::SendMessage(_hParent, NPPM_DMMREGASDCKDLG, 0, (LPARAM)&data); + ::SendMessage(_hParent, NPPM_DMMREGASDCKDLG, 0, reinterpret_cast(&data)); pFinder->_scintView.init(_hInst, pFinder->getHSelf()); // Subclass the ScintillaEditView for the Finder (Scintilla doesn't notify all key presses) - originalFinderProc = SetWindowLongPtr(pFinder->_scintView.getHSelf(), GWLP_WNDPROC, (LONG_PTR)finderProc); + originalFinderProc = SetWindowLongPtr(pFinder->_scintView.getHSelf(), GWLP_WNDPROC, reinterpret_cast(finderProc)); pFinder->setFinderReadOnly(true); pFinder->_scintView.execute(SCI_SETCODEPAGE, SC_CP_UTF8); @@ -2075,47 +2075,24 @@ Finder * FindReplaceDlg::createFinder() pFinder->_scintView.display(); pFinder->display(); - //::SendMessage(_hParent, NPPM_DMMHIDE, 0, (LPARAM)pFinder->getHSelf()); ::UpdateWindow(_hParent); - //justCreated = true; pFinder->setFinderStyle(); // Send the address of _MarkingsStruct to the lexer char ptrword[sizeof(void*) * 2 + 1]; sprintf(ptrword, "%p", &pFinder->_markingsStruct); - pFinder->_scintView.execute(SCI_SETPROPERTY, (WPARAM)"@MarkingsStruct", (LPARAM)ptrword); + pFinder->_scintView.execute(SCI_SETPROPERTY, reinterpret_cast("@MarkingsStruct"), reinterpret_cast(ptrword)); _findersOfFinder.push_back(pFinder); ::SendMessage(pFinder->getHSelf(), WM_SIZE, 0, 0); // Show finder - ::SendMessage(_hParent, NPPM_DMMSHOW, 0, (LPARAM)pFinder->getHSelf()); + ::SendMessage(_hParent, NPPM_DMMSHOW, 0, reinterpret_cast(pFinder->getHSelf())); pFinder->_scintView.getFocus(); return pFinder; - - /* - { - if (_findAllResult == 1) - wsprintf(_findAllResultStr, TEXT("1 hit")); - else - wsprintf(_findAllResultStr, TEXT("%d hits"), _findAllResult); - if (_findAllResult) - { - focusOnFinder(); - } - else - { - // Show finder - ::SendMessage(_hParent, NPPM_DMMSHOW, 0, (LPARAM)pFinder->getHSelf()); - getFocus(); // no hits - } - } - else // error - search folder doesn't exist - ::SendMessage(_hSelf, WM_NEXTDLGCTL, (WPARAM)::GetDlgItem(_hSelf, IDD_FINDINFILES_DIR_COMBO), TRUE); - */ } bool FindReplaceDlg::removeFinder(Finder *finder2remove) @@ -2137,7 +2114,7 @@ void FindReplaceDlg::setSearchText(TCHAR * txt2find) { if (txt2find && txt2find[0]) { // We got a valid search string - ::SendMessage(hCombo, CB_SETCURSEL, (WPARAM)-1, 0); // remove selection - to allow using down arrow to get to last searched word + ::SendMessage(hCombo, CB_SETCURSEL, static_cast(-1), 0); // remove selection - to allow using down arrow to get to last searched word ::SetDlgItemText(_hSelf, IDFINDWHAT, txt2find); } ::SendMessage(hCombo, CB_SETEDITSEL, 0, MAKELPARAM(0, -1)); // select all text - fast edit @@ -2254,11 +2231,11 @@ void FindReplaceDlg::saveInMacro(size_t cmd, int cmdType) booleans |= _options._doMarkLine?IDF_MARKLINE_CHECK:0; } if (cmdType & FR_OP_REPLACE) - ::SendMessage(_hParent, WM_FRSAVE_STR, IDREPLACEWITH, (LPARAM)_options._str4Replace.c_str()); + ::SendMessage(_hParent, WM_FRSAVE_STR, IDREPLACEWITH, reinterpret_cast(_options._str4Replace.c_str())); if (cmdType & FR_OP_FIF) { - ::SendMessage(_hParent, WM_FRSAVE_STR, IDD_FINDINFILES_DIR_COMBO, (LPARAM)_options._directory.c_str()); - ::SendMessage(_hParent, WM_FRSAVE_STR, IDD_FINDINFILES_FILTERS_COMBO, (LPARAM)_options._filters.c_str()); + ::SendMessage(_hParent, WM_FRSAVE_STR, IDD_FINDINFILES_DIR_COMBO, reinterpret_cast(_options._directory.c_str())); + ::SendMessage(_hParent, WM_FRSAVE_STR, IDD_FINDINFILES_FILTERS_COMBO, reinterpret_cast(_options._filters.c_str())); booleans |= _options._isRecursive?IDF_FINDINFILES_RECURSIVE_CHECK:0; booleans |= _options._isInHiddenDir?IDF_FINDINFILES_INHIDDENDIR_CHECK:0; } @@ -2776,7 +2753,7 @@ void Finder::openAll() for (size_t i = 0; i < sz; ++i) { - ::SendMessage(::GetParent(_hParent), WM_DOOPEN, 0, (LPARAM)_pMainFoundInfos->at(i)._fullPath.c_str()); + ::SendMessage(::GetParent(_hParent), WM_DOOPEN, 0, reinterpret_cast(_pMainFoundInfos->at(i)._fullPath.c_str())); } } diff --git a/PowerEditor/src/ScitillaComponent/UserDefineDialog.cpp b/PowerEditor/src/ScitillaComponent/UserDefineDialog.cpp index bb03ef0b..abf1611a 100644 --- a/PowerEditor/src/ScitillaComponent/UserDefineDialog.cpp +++ b/PowerEditor/src/ScitillaComponent/UserDefineDialog.cpp @@ -1579,7 +1579,7 @@ INT_PTR CALLBACK StylerDlg::dlgProc(HWND hwnd, UINT message, WPARAM wParam, LPAR // for the font size combo HWND hFontSizeCombo = ::GetDlgItem(hwnd, IDC_STYLER_COMBO_FONT_SIZE); for(int j = 0 ; j < int(sizeof(fontSizeStrs))/(3*sizeof(TCHAR)) ; ++j) - ::SendMessage(hFontSizeCombo, CB_ADDSTRING, 0, (LPARAM)fontSizeStrs[j]); + ::SendMessage(hFontSizeCombo, CB_ADDSTRING, 0, reinterpret_cast(fontSizeStrs[j])); TCHAR size[10]; if (style._fontSize == -1) @@ -1587,7 +1587,7 @@ INT_PTR CALLBACK StylerDlg::dlgProc(HWND hwnd, UINT message, WPARAM wParam, LPAR else wsprintf(size, TEXT("%d"),style._fontSize); - auto i = ::SendMessage(hFontSizeCombo, CB_FINDSTRINGEXACT, (WPARAM)-1, (LPARAM)size); + auto i = ::SendMessage(hFontSizeCombo, CB_FINDSTRINGEXACT, static_cast(-1), reinterpret_cast(size)); if (i != CB_ERR) ::SendMessage(hFontSizeCombo, CB_SETCURSEL, i, 0); @@ -1596,11 +1596,11 @@ INT_PTR CALLBACK StylerDlg::dlgProc(HWND hwnd, UINT message, WPARAM wParam, LPAR const std::vector & fontlist = pNppParam->getFontList(); for (size_t j = 0, len = fontlist.size() ; j < len ; ++j) { - auto k = ::SendMessage(hFontNameCombo, CB_ADDSTRING, 0, (LPARAM)fontlist[j].c_str()); - ::SendMessage(hFontNameCombo, CB_SETITEMDATA, k, (LPARAM)fontlist[j].c_str()); + auto k = ::SendMessage(hFontNameCombo, CB_ADDSTRING, 0, reinterpret_cast(fontlist[j].c_str())); + ::SendMessage(hFontNameCombo, CB_SETITEMDATA, k, reinterpret_cast(fontlist[j].c_str())); } - i = ::SendMessage(hFontNameCombo, CB_FINDSTRINGEXACT, (WPARAM)-1, (LPARAM)style._fontName); + i = ::SendMessage(hFontNameCombo, CB_FINDSTRINGEXACT, static_cast(-1), reinterpret_cast(style._fontName)); if (i == CB_ERR) i = 0; ::SendMessage(hFontNameCombo, CB_SETCURSEL, i, 0); diff --git a/PowerEditor/src/ScitillaComponent/columnEditor.cpp b/PowerEditor/src/ScitillaComponent/columnEditor.cpp index 146d58c7..2d26e0c9 100644 --- a/PowerEditor/src/ScitillaComponent/columnEditor.cpp +++ b/PowerEditor/src/ScitillaComponent/columnEditor.cpp @@ -87,7 +87,7 @@ INT_PTR CALLBACK ColumnEditorDlg::run_dlgProc(UINT message, WPARAM wParam, LPARA if (isTextMode) { - ::SendDlgItemMessage(_hSelf, IDC_COL_TEXT_EDIT, WM_GETTEXT, stringSize, (LPARAM)str); + ::SendDlgItemMessage(_hSelf, IDC_COL_TEXT_EDIT, WM_GETTEXT, stringSize, reinterpret_cast(str)); display(false); diff --git a/PowerEditor/src/WinControls/AboutDlg/AboutDlg.cpp b/PowerEditor/src/WinControls/AboutDlg/AboutDlg.cpp index f85ed019..33a3e53d 100644 --- a/PowerEditor/src/WinControls/AboutDlg/AboutDlg.cpp +++ b/PowerEditor/src/WinControls/AboutDlg/AboutDlg.cpp @@ -51,11 +51,11 @@ INT_PTR CALLBACK AboutDlg::run_dlgProc(UINT message, WPARAM wParam, LPARAM lPara LPCTSTR bitness = pNppParam ->isx64() ? TEXT("(64-bit)") : TEXT("(32-bit)"); ::SetDlgItemText(_hSelf, IDC_VERSION_BIT, bitness); - ::SendMessage(compileDateHandle, WM_SETTEXT, 0, (LPARAM)buildTime.c_str()); + ::SendMessage(compileDateHandle, WM_SETTEXT, 0, reinterpret_cast(buildTime.c_str())); ::EnableWindow(compileDateHandle, FALSE); HWND licenceEditHandle = ::GetDlgItem(_hSelf, IDC_LICENCE_EDIT); - ::SendMessage(licenceEditHandle, WM_SETTEXT, 0, (LPARAM)LICENCE_TXT); + ::SendMessage(licenceEditHandle, WM_SETTEXT, 0, reinterpret_cast(LICENCE_TXT)); _emailLink.init(_hInst, _hSelf); //_emailLink.create(::GetDlgItem(_hSelf, IDC_AUTHOR_NAME), TEXT("mailto:don.h@free.fr")); diff --git a/PowerEditor/src/WinControls/AnsiCharPanel/ansiCharPanel.cpp b/PowerEditor/src/WinControls/AnsiCharPanel/ansiCharPanel.cpp index 32dcef39..cdb8cd8d 100644 --- a/PowerEditor/src/WinControls/AnsiCharPanel/ansiCharPanel.cpp +++ b/PowerEditor/src/WinControls/AnsiCharPanel/ansiCharPanel.cpp @@ -132,8 +132,8 @@ void AnsiCharPanel::insertChar(unsigned char char2insert) const MultiByteToWideChar(codepage, 0, charStr, -1, wCharStr, sizeof(wCharStr)); WideCharToMultiByte(CP_UTF8, 0, wCharStr, -1, multiByteStr, sizeof(multiByteStr), NULL, NULL); } - (*_ppEditView)->execute(SCI_REPLACESEL, 0, (LPARAM)""); - size_t len = (char2insert < 128)?1:strlen(multiByteStr); - (*_ppEditView)->execute(SCI_ADDTEXT, len, (LPARAM)multiByteStr); + (*_ppEditView)->execute(SCI_REPLACESEL, 0, reinterpret_cast("")); + size_t len = (char2insert < 128) ? 1 : strlen(multiByteStr); + (*_ppEditView)->execute(SCI_ADDTEXT, len, reinterpret_cast(multiByteStr)); (*_ppEditView)->getFocus(); } \ No newline at end of file diff --git a/PowerEditor/src/WinControls/ClipboardHistory/clipboardHistoryPanel.cpp b/PowerEditor/src/WinControls/ClipboardHistory/clipboardHistoryPanel.cpp index 7e263386..b3cae223 100644 --- a/PowerEditor/src/WinControls/ClipboardHistory/clipboardHistoryPanel.cpp +++ b/PowerEditor/src/WinControls/ClipboardHistory/clipboardHistoryPanel.cpp @@ -173,7 +173,7 @@ void ClipboardHistoryPanel::addToClipboadHistory(ClipboardData cbd) StringArray sa(cbd, MAX_DISPLAY_LENGTH); TCHAR *displayStr = (TCHAR *)sa.getPointer(); - ::SendDlgItemMessage(_hSelf, IDC_LIST_CLIPBOARD, LB_INSERTSTRING, 0, (LPARAM)displayStr); + ::SendDlgItemMessage(_hSelf, IDC_LIST_CLIPBOARD, LB_INSERTSTRING, 0, reinterpret_cast(displayStr)); } @@ -254,8 +254,8 @@ INT_PTR CALLBACK ClipboardHistoryPanel::run_dlgProc(UINT message, WPARAM wParam, char *c = new char[nbChar+1]; WideCharToMultiByte(codepage, 0, (wchar_t *)ba.getPointer(), static_cast(ba.getLength()), c, nbChar + 1, NULL, NULL); - (*_ppEditView)->execute(SCI_REPLACESEL, 0, (LPARAM)""); - (*_ppEditView)->execute(SCI_ADDTEXT, strlen(c), (LPARAM)c); + (*_ppEditView)->execute(SCI_REPLACESEL, 0, reinterpret_cast("")); + (*_ppEditView)->execute(SCI_ADDTEXT, strlen(c), reinterpret_cast(c)); (*_ppEditView)->getFocus(); delete [] c; } @@ -277,13 +277,13 @@ INT_PTR CALLBACK ClipboardHistoryPanel::run_dlgProc(UINT message, WPARAM wParam, case WM_CTLCOLORLISTBOX: { if (_lbBgColor != -1) - return (LRESULT)::CreateSolidBrush((COLORREF)_lbBgColor); + return reinterpret_cast(::CreateSolidBrush(_lbBgColor)); break; } case WM_DRAWITEM: { - drawItem((DRAWITEMSTRUCT *)lParam); + drawItem(reinterpret_cast(lParam)); break; } default : diff --git a/PowerEditor/src/WinControls/ColourPicker/ColourPicker.cpp b/PowerEditor/src/WinControls/ColourPicker/ColourPicker.cpp index 8a0f2f0f..7301ebad 100644 --- a/PowerEditor/src/WinControls/ColourPicker/ColourPicker.cpp +++ b/PowerEditor/src/WinControls/ColourPicker/ColourPicker.cpp @@ -138,7 +138,7 @@ LRESULT ColourPicker::runProc(UINT Message, WPARAM wParam, LPARAM lParam) { _isEnabled = !_isEnabled; redraw(); - ::SendMessage(_hParent, WM_COMMAND, MAKELONG(0, CPN_COLOURPICKED), (LPARAM)_hSelf); + ::SendMessage(_hParent, WM_COMMAND, MAKELONG(0, CPN_COLOURPICKED), reinterpret_cast(_hSelf)); break; } @@ -164,7 +164,7 @@ LRESULT ColourPicker::runProc(UINT Message, WPARAM wParam, LPARAM lParam) redraw(); _pColourPopup->display(false); - ::SendMessage(_hParent, WM_COMMAND, MAKELONG(0, CPN_COLOURPICKED), (LPARAM)_hSelf); + ::SendMessage(_hParent, WM_COMMAND, MAKELONG(0, CPN_COLOURPICKED), reinterpret_cast(_hSelf)); return TRUE; } diff --git a/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.cpp b/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.cpp index dafd4404..1e8b9c13 100644 --- a/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.cpp +++ b/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.cpp @@ -109,7 +109,7 @@ INT_PTR CALLBACK WordStyleDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM l for(size_t i = 0 ; i < themeSwitcher.size() ; ++i) { pair & themeInfo = themeSwitcher.getElementFromIndex(i); - int j = static_cast(::SendMessage(_hSwitch2ThemeCombo, CB_ADDSTRING, 0, (LPARAM)themeInfo.first.c_str())); + int j = static_cast(::SendMessage(_hSwitch2ThemeCombo, CB_ADDSTRING, 0, reinterpret_cast(themeInfo.first.c_str()))); if (! themeInfo.second.compare( nppParamInst->getNppGUI()._themeName ) ) { _currentThemeIndex = j; @@ -127,13 +127,13 @@ INT_PTR CALLBACK WordStyleDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM l ::SendMessage(_hSwitch2ThemeCombo, CB_SETCURSEL, _currentThemeIndex, 0); for(int i = 0 ; i < sizeof(fontSizeStrs)/(3*sizeof(TCHAR)) ; ++i) - ::SendMessage(_hFontSizeCombo, CB_ADDSTRING, 0, (LPARAM)fontSizeStrs[i]); + ::SendMessage(_hFontSizeCombo, CB_ADDSTRING, 0, reinterpret_cast(fontSizeStrs[i])); const std::vector & fontlist = (NppParameters::getInstance())->getFontList(); for (size_t i = 0, len = fontlist.size() ; i < len ; ++i) { - auto j = ::SendMessage(_hFontNameCombo, CB_ADDSTRING, 0, (LPARAM)fontlist[i].c_str()); - ::SendMessage(_hFontNameCombo, CB_SETITEMDATA, j, (LPARAM)fontlist[i].c_str()); + auto j = ::SendMessage(_hFontNameCombo, CB_ADDSTRING, 0, reinterpret_cast(fontlist[i].c_str())); + ::SendMessage(_hFontNameCombo, CB_SETITEMDATA, j, reinterpret_cast(fontlist[i].c_str())); } _pFgColour = new ColourPicker; diff --git a/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.h b/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.h index 67e8680e..6b4c1ba7 100644 --- a/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.h +++ b/PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.h @@ -115,7 +115,7 @@ public : NppParameters *nppParamInst = NppParameters::getInstance(); ThemeSwitcher & themeSwitcher = nppParamInst->getThemeSwitcher(); std::pair & themeInfo = themeSwitcher.getElementFromIndex(themeSwitcher.size() - 1); - ::SendMessage(_hSwitch2ThemeCombo, CB_ADDSTRING, 0, (LPARAM)themeInfo.first.c_str()); + ::SendMessage(_hSwitch2ThemeCombo, CB_ADDSTRING, 0, reinterpret_cast(themeInfo.first.c_str())); }; diff --git a/PowerEditor/src/WinControls/DockingWnd/DockingCont.cpp b/PowerEditor/src/WinControls/DockingWnd/DockingCont.cpp index fd2c85cf..a2161ee8 100644 --- a/PowerEditor/src/WinControls/DockingWnd/DockingCont.cpp +++ b/PowerEditor/src/WinControls/DockingWnd/DockingCont.cpp @@ -240,7 +240,7 @@ tTbData* DockingCont::getDataOfActiveTb() TCITEM tcItem = {0}; tcItem.mask = TCIF_PARAM; - ::SendMessage(_hContTab, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); pTbData = (tTbData*)tcItem.lParam; } @@ -257,7 +257,7 @@ vector DockingCont::getDataOfVisTb() for(int iItem = 0; iItem < iItemCnt; ++iItem) { - ::SendMessage(_hContTab, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); vTbData.push_back((tTbData*)tcItem.lParam); } return vTbData; @@ -272,7 +272,7 @@ bool DockingCont::isTbVis(tTbData* data) for(int iItem = 0; iItem < iItemCnt; ++iItem) { - ::SendMessage(_hContTab, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); if (!tcItem.lParam) return false; if (((tTbData*)tcItem.lParam) == data) @@ -747,12 +747,12 @@ LRESULT DockingCont::runProcTab(HWND hwnd, UINT Message, WPARAM wParam, LPARAM l // get text of toolbar tcItem.mask = TCIF_PARAM; - ::SendMessage(hwnd, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(hwnd, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); if (!tcItem.lParam) return FALSE; toolTip.init(_hInst, hwnd); - toolTip.Show(rc, ((tTbData*)tcItem.lParam)->pszName, info.pt.x, info.pt.y + 20); + toolTip.Show(rc, (reinterpret_cast(tcItem.lParam))->pszName, info.pt.x, info.pt.y + 20); } } @@ -774,14 +774,14 @@ LRESULT DockingCont::runProcTab(HWND hwnd, UINT Message, WPARAM wParam, LPARAM l // get selected sub item info.pt.x = LOWORD(lParam); info.pt.y = HIWORD(lParam); - iItem = static_cast(::SendMessage(hwnd, TCM_HITTEST, 0, (LPARAM)&info)); + iItem = static_cast(::SendMessage(hwnd, TCM_HITTEST, 0, reinterpret_cast(&info))); // recalc mouse position ::ClientToScreen(hwnd, &info.pt); // get text of toolbar tcItem.mask = TCIF_PARAM; - ::SendMessage(hwnd, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(hwnd, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); if (!tcItem.lParam) return FALSE; @@ -809,7 +809,7 @@ LRESULT DockingCont::runProcTab(HWND hwnd, UINT Message, WPARAM wParam, LPARAM l // get selected sub item info.pt.x = LOWORD(lParam); info.pt.y = HIWORD(lParam); - iItem = static_cast(::SendMessage(hwnd, TCM_HITTEST, 0, (LPARAM)&info)); + iItem = static_cast(::SendMessage(hwnd, TCM_HITTEST, 0, reinterpret_cast(&info))); SelectTab(iItem); } @@ -832,12 +832,12 @@ void DockingCont::drawTabItem(DRAWITEMSTRUCT *pDrawItemStruct) // get current selected item tcItem.mask = TCIF_PARAM; - ::SendMessage(_hContTab, TCM_GETITEM, nTab, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_GETITEM, nTab, reinterpret_cast(&tcItem)); if (!tcItem.lParam) return; - const TCHAR *text = ((tTbData*)tcItem.lParam)->pszName; - int length = lstrlen(((tTbData*)tcItem.lParam)->pszName); + const TCHAR *text = reinterpret_cast(tcItem.lParam)->pszName; + int length = lstrlen(reinterpret_cast(tcItem.lParam)->pszName); // get drawing context @@ -1247,7 +1247,7 @@ void DockingCont::viewToolbar(tTbData *pTbData) UINT iItem = getActiveTb(); tcItem.mask = TCIF_PARAM; - ::SendMessage(_hContTab, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); if (!tcItem.lParam) return; @@ -1263,13 +1263,13 @@ void DockingCont::viewToolbar(tTbData *pTbData) if (iTabPos == -1) { // set only params and text even if icon available - ::SendMessage(_hContTab, TCM_INSERTITEM, iItemCnt, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_INSERTITEM, iItemCnt, reinterpret_cast(&tcItem)); SelectTab(iItemCnt); } // if exists select it and update data else { - ::SendMessage(_hContTab, TCM_SETITEM, iTabPos, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_SETITEM, iTabPos, reinterpret_cast(&tcItem)); SelectTab(iTabPos); } @@ -1385,7 +1385,7 @@ void DockingCont::SelectTab(int iTab) szText += pszMaxTxt; } tcItem.pszText = (TCHAR *)szText.c_str(); - ::SendMessage(_hContTab, TCM_SETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_SETITEM, iItem, reinterpret_cast(&tcItem)); } // selects the pressed tab and store previous tab @@ -1412,7 +1412,7 @@ bool DockingCont::updateCaption() // get data of new active dialog tcItem.mask = TCIF_PARAM; - ::SendMessage(_hContTab, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); if (!tcItem.lParam) return false; @@ -1447,7 +1447,7 @@ void DockingCont::focusClient() { // get data of new active dialog tcItem.mask = TCIF_PARAM; - ::SendMessage(_hContTab, TCM_GETITEM, iItem, (LPARAM)&tcItem); + ::SendMessage(_hContTab, TCM_GETITEM, iItem, reinterpret_cast(&tcItem)); // set focus if (!tcItem.lParam) diff --git a/PowerEditor/src/WinControls/DockingWnd/DockingManager.cpp b/PowerEditor/src/WinControls/DockingWnd/DockingManager.cpp index 67ea60e3..ffe504ac 100644 --- a/PowerEditor/src/WinControls/DockingWnd/DockingManager.cpp +++ b/PowerEditor/src/WinControls/DockingWnd/DockingManager.cpp @@ -229,12 +229,12 @@ LRESULT DockingManager::runProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM l // activate/deactivate titlebar of toolbars for (size_t iCont = DOCKCONT_MAX, len = _vContainer.size(); iCont < len; ++iCont) { - ::SendMessage(_vContainer[iCont]->getHSelf(), WM_NCACTIVATE, wParam, (LPARAM)-1); + ::SendMessage(_vContainer[iCont]->getHSelf(), WM_NCACTIVATE, wParam, static_cast(-1)); } if (static_cast(lParam) != -1) { - ::SendMessage(_hParent, WM_NCACTIVATE, wParam, (LPARAM)-1); + ::SendMessage(_hParent, WM_NCACTIVATE, wParam, static_cast(-1)); } break; } @@ -376,7 +376,7 @@ LRESULT DockingManager::runProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM l } case DMM_GETIMAGELIST: { - return (LPARAM)_hImageList; + return reinterpret_cast(_hImageList); } case DMM_GETICONPOS: { @@ -714,7 +714,7 @@ LRESULT DockingManager::SendNotify(HWND hWnd, UINT message) nmhdr.code = message; nmhdr.hwndFrom = _hParent; nmhdr.idFrom = ::GetDlgCtrlID(_hParent); - ::SendMessage(hWnd, WM_NOTIFY, nmhdr.idFrom, (LPARAM)&nmhdr); + ::SendMessage(hWnd, WM_NOTIFY, nmhdr.idFrom, reinterpret_cast(&nmhdr)); return ::GetWindowLongPtr(hWnd, DWLP_MSGRESULT); } diff --git a/PowerEditor/src/WinControls/DocumentMap/documentMap.cpp b/PowerEditor/src/WinControls/DocumentMap/documentMap.cpp index b47fefd9..36ddc710 100644 --- a/PowerEditor/src/WinControls/DocumentMap/documentMap.cpp +++ b/PowerEditor/src/WinControls/DocumentMap/documentMap.cpp @@ -35,7 +35,7 @@ void DocumentMap::reloadMap() if (_pScintillaEditView && _ppEditView) { Document currentDoc = (*_ppEditView)->execute(SCI_GETDOCPOINTER); - _pScintillaEditView->execute(SCI_SETDOCPOINTER, 0, (LPARAM)currentDoc); + _pScintillaEditView->execute(SCI_SETDOCPOINTER, 0, static_cast(currentDoc)); // // sync with the current document @@ -271,9 +271,9 @@ INT_PTR CALLBACK DocumentMap::run_dlgProc(UINT message, WPARAM wParam, LPARAM lP { case WM_INITDIALOG : { - HWND hwndScintilla = (HWND)::SendMessage(_hParent, NPPM_CREATESCINTILLAHANDLE, 0, (LPARAM)_hSelf); - _pScintillaEditView = (ScintillaEditView *)::SendMessage(_hParent, NPPM_INTERNAL_GETSCINTEDTVIEW, 0, (LPARAM)hwndScintilla); - _pScintillaEditView->execute(SCI_SETZOOM, (WPARAM)-10, 0); + HWND hwndScintilla = reinterpret_cast(::SendMessage(_hParent, NPPM_CREATESCINTILLAHANDLE, 0, reinterpret_cast(_hSelf))); + _pScintillaEditView = reinterpret_cast(::SendMessage(_hParent, NPPM_INTERNAL_GETSCINTEDTVIEW, 0, reinterpret_cast(hwndScintilla))); + _pScintillaEditView->execute(SCI_SETZOOM, static_cast(-10), 0); _pScintillaEditView->execute(SCI_SETVSCROLLBAR, FALSE, 0); _pScintillaEditView->execute(SCI_SETHSCROLLBAR, FALSE, 0); diff --git a/PowerEditor/src/WinControls/FileBrowser/fileBrowser.cpp b/PowerEditor/src/WinControls/FileBrowser/fileBrowser.cpp index 79616937..84953df2 100644 --- a/PowerEditor/src/WinControls/FileBrowser/fileBrowser.cpp +++ b/PowerEditor/src/WinControls/FileBrowser/fileBrowser.cpp @@ -400,7 +400,7 @@ void FileBrowser::openSelectFile() if (::PathIsDirectory(fullPath.c_str())) return; - ::SendMessage(_hParent, NPPM_DOOPEN, 0, (LPARAM)(fullPath.c_str())); + ::SendMessage(_hParent, NPPM_DOOPEN, 0, reinterpret_cast(fullPath.c_str())); } @@ -439,7 +439,7 @@ void FileBrowser::notified(LPNMHDR notification) { // Get the old label tvItem.hItem = _treeView.getSelection(); - ::SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0,(LPARAM)&tvItem); + ::SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0, reinterpret_cast(&tvItem)); size_t len = lstrlen(tvItem.pszText); // Find the position of old label in File path @@ -712,7 +712,7 @@ void FileBrowser::popupMenuCmd(int cmdID) { if (not selectedNode) return; generic_string path = getNodePath(selectedNode); - ::SendMessage(_hParent, NPPM_LAUNCHFINDINFILESDLG, (WPARAM)path.c_str(), 0); + ::SendMessage(_hParent, NPPM_LAUNCHFINDINFILESDLG, reinterpret_cast(path.c_str()), 0); } break; @@ -1009,7 +1009,7 @@ HTREEITEM FileBrowser::getRootFromFullPath(const generic_string & rootPath) cons tvItem.mask = TVIF_PARAM; tvItem.cchTextMax = MAX_PATH; tvItem.hItem = hItemNode; - SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0, (LPARAM)&tvItem); + SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0, reinterpret_cast(&tvItem)); if (tvItem.lParam != 0 && rootPath == *((generic_string *)tvItem.lParam)) node = hItemNode; @@ -1031,7 +1031,7 @@ HTREEITEM FileBrowser::findChildNodeFromName(HTREEITEM parent, generic_string la tvItem.pszText = textBuffer; tvItem.cchTextMax = MAX_PATH; tvItem.hItem = hItemNode; - SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0, (LPARAM)&tvItem); + SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0, reinterpret_cast(&tvItem)); if (label == tvItem.pszText) { @@ -1054,7 +1054,7 @@ vector FileBrowser::getRoots() const tvItem.mask = TVIF_PARAM; tvItem.cchTextMax = MAX_PATH; tvItem.hItem = hItemNode; - SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0, (LPARAM)&tvItem); + SendMessage(_treeView.getHSelf(), TVM_GETITEM, 0, reinterpret_cast(&tvItem)); roots.push_back(*((generic_string *)tvItem.lParam)); } @@ -1497,14 +1497,14 @@ DWORD WINAPI FolderUpdater::watching(void *params) case FILE_ACTION_ADDED: file2Change.push_back(wstrFilename.GetString()); //thisFolderUpdater->updateTree(dwAction, file2Change); - ::SendMessage((thisFolderUpdater->_pFileBrowser)->getHSelf(), FB_ADDFILE, (WPARAM)nullptr, (LPARAM)&file2Change); + ::SendMessage((thisFolderUpdater->_pFileBrowser)->getHSelf(), FB_ADDFILE, reinterpret_cast(nullptr), reinterpret_cast(&file2Change)); oldName = TEXT(""); break; case FILE_ACTION_REMOVED: file2Change.push_back(wstrFilename.GetString()); //thisFolderUpdater->updateTree(dwAction, file2Change); - ::SendMessage((thisFolderUpdater->_pFileBrowser)->getHSelf(), FB_RMFILE, (WPARAM)nullptr, (LPARAM)&file2Change); + ::SendMessage((thisFolderUpdater->_pFileBrowser)->getHSelf(), FB_RMFILE, reinterpret_cast(nullptr), reinterpret_cast(&file2Change)); oldName = TEXT(""); break; @@ -1522,7 +1522,7 @@ DWORD WINAPI FolderUpdater::watching(void *params) file2Change.push_back(oldName); file2Change.push_back(wstrFilename.GetString()); //thisFolderUpdater->updateTree(dwAction, file2Change); - ::SendMessage((thisFolderUpdater->_pFileBrowser)->getHSelf(), FB_RNFILE, (WPARAM)nullptr, (LPARAM)&file2Change); + ::SendMessage((thisFolderUpdater->_pFileBrowser)->getHSelf(), FB_RNFILE, reinterpret_cast(nullptr), reinterpret_cast(&file2Change)); } oldName = TEXT(""); break; diff --git a/PowerEditor/src/WinControls/FunctionList/functionListPanel.cpp b/PowerEditor/src/WinControls/FunctionList/functionListPanel.cpp index c81514cb..a03bb004 100644 --- a/PowerEditor/src/WinControls/FunctionList/functionListPanel.cpp +++ b/PowerEditor/src/WinControls/FunctionList/functionListPanel.cpp @@ -199,7 +199,7 @@ void FunctionListPanel::sortOrUnsort() else { TCHAR text2search[MAX_PATH] ; - ::SendMessage(_hSearchEdit, WM_GETTEXT, MAX_PATH, (LPARAM)text2search); + ::SendMessage(_hSearchEdit, WM_GETTEXT, MAX_PATH, reinterpret_cast(text2search)); if (text2search[0] == '\0') // main view { @@ -288,13 +288,13 @@ void FunctionListPanel::reload() TreeParams *previousParams = getFromStateArray(fullFilePath); if (!previousParams) { - ::SendMessage(_hSearchEdit, WM_SETTEXT, 0, (LPARAM)TEXT("")); + ::SendMessage(_hSearchEdit, WM_SETTEXT, 0, reinterpret_cast(TEXT(""))); setSort(false); _treeView.expand(root); } else { - ::SendMessage(_hSearchEdit, WM_SETTEXT, 0, (LPARAM)(previousParams->_searchParameters)._text2Find.c_str()); + ::SendMessage(_hSearchEdit, WM_SETTEXT, 0, reinterpret_cast((previousParams->_searchParameters)._text2Find.c_str())); _treeView.restoreFoldingStateFrom(previousParams->_treeState, root); @@ -469,7 +469,7 @@ BOOL FunctionListPanel::setTreeViewImageList(int root_id, int node_id, int leaf_ void FunctionListPanel::searchFuncAndSwitchView() { TCHAR text2search[MAX_PATH] ; - ::SendMessage(_hSearchEdit, WM_GETTEXT, MAX_PATH, (LPARAM)text2search); + ::SendMessage(_hSearchEdit, WM_GETTEXT, MAX_PATH, reinterpret_cast(text2search)); bool doSort = shouldSort(); if (text2search[0] == '\0') @@ -574,16 +574,15 @@ INT_PTR CALLBACK FunctionListPanel::run_dlgProc(UINT message, WPARAM wParam, LPA _hToolbarMenu = CreateWindowEx(0,TOOLBARCLASSNAME,NULL, style, 0,0,0,0,_hSelf,(HMENU)0, _hInst, NULL); - //::GetWindowLongPtr(_hToolbarMenu, GWLP_WNDPROC); - oldFunclstToolbarProc = (WNDPROC)::SetWindowLongPtr(_hToolbarMenu, GWLP_WNDPROC, (LONG_PTR)funclstToolbarProc); + oldFunclstToolbarProc = reinterpret_cast(::SetWindowLongPtr(_hToolbarMenu, GWLP_WNDPROC, reinterpret_cast(funclstToolbarProc))); TBBUTTON tbButtons[3]; // Add the bmap image into toolbar's imagelist TBADDBITMAP addbmp = {_hInst, 0}; addbmp.nID = IDI_FUNCLIST_SORTBUTTON; - ::SendMessage(_hToolbarMenu, TB_ADDBITMAP, 1, (LPARAM)&addbmp); + ::SendMessage(_hToolbarMenu, TB_ADDBITMAP, 1, reinterpret_cast(&addbmp)); addbmp.nID = IDI_FUNCLIST_RELOADBUTTON; - ::SendMessage(_hToolbarMenu, TB_ADDBITMAP, 1, (LPARAM)&addbmp); + ::SendMessage(_hToolbarMenu, TB_ADDBITMAP, 1, reinterpret_cast(&addbmp)); // Place holder of search text field tbButtons[0].idCommand = 0; @@ -606,7 +605,7 @@ INT_PTR CALLBACK FunctionListPanel::run_dlgProc(UINT message, WPARAM wParam, LPA ::SendMessage(_hToolbarMenu, TB_BUTTONSTRUCTSIZE, (WPARAM)sizeof(TBBUTTON), 0); ::SendMessage(_hToolbarMenu, TB_SETBUTTONSIZE, (WPARAM)0, (LPARAM)MAKELONG(16, 16)); - ::SendMessage(_hToolbarMenu, TB_ADDBUTTONS, (WPARAM)sizeof(tbButtons) / sizeof(TBBUTTON), (LPARAM)&tbButtons); + ::SendMessage(_hToolbarMenu, TB_ADDBUTTONS, (WPARAM)sizeof(tbButtons) / sizeof(TBBUTTON), reinterpret_cast(&tbButtons)); ::SendMessage(_hToolbarMenu, TB_AUTOSIZE, 0, 0); ShowWindow(_hToolbarMenu, SW_SHOW); diff --git a/PowerEditor/src/WinControls/Grid/BabyGrid.cpp b/PowerEditor/src/WinControls/Grid/BabyGrid.cpp index 06f225b7..1abac7be 100644 --- a/PowerEditor/src/WinControls/Grid/BabyGrid.cpp +++ b/PowerEditor/src/WinControls/Grid/BabyGrid.cpp @@ -1690,7 +1690,7 @@ LRESULT CALLBACK GridProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam) int j = static_cast(SendMessage(BGHS[SelfIndex].hlist1, LB_GETCOUNT, 0, 0)); if(j>0) { - SendMessage(BGHS[SelfIndex].hlist1,LB_GETTEXT,j-1,(LPARAM)buffer); + SendMessage(BGHS[SelfIndex].hlist1, LB_GETTEXT, j - 1, reinterpret_cast(buffer)); buffer[5]=0x00; j=generic_atoi(buffer); if(j>SendMessage(hWnd,BGM_GETROWS,0,0)) diff --git a/PowerEditor/src/WinControls/Grid/ShortcutMapper.cpp b/PowerEditor/src/WinControls/Grid/ShortcutMapper.cpp index 0404e684..bc86a7bf 100644 --- a/PowerEditor/src/WinControls/Grid/ShortcutMapper.cpp +++ b/PowerEditor/src/WinControls/Grid/ShortcutMapper.cpp @@ -332,9 +332,9 @@ INT_PTR CALLBACK ShortcutMapper::run_dlgProc(UINT message, WPARAM wParam, LPARAM const bool isConflict = findKeyConflicts(&conflictInfo, *reinterpret_cast(wParam), _babygrid.getSelectedRow() - 1); *reinterpret_cast(lParam) = isConflict; if (isConflict) - ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, (LPARAM)conflictInfo.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, reinterpret_cast(conflictInfo.c_str())); else - ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, (LPARAM)_assignInfo.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, reinterpret_cast(_assignInfo.c_str())); return TRUE; } @@ -724,9 +724,9 @@ INT_PTR CALLBACK ShortcutMapper::run_dlgProc(UINT message, WPARAM wParam, LPARAM } if (conflictInfo.empty()) - ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, (LPARAM)_defaultInfo.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, reinterpret_cast(_defaultInfo.c_str())); else - ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, (LPARAM)conflictInfo.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_BABYGRID_INFO, WM_SETTEXT, 0, reinterpret_cast(conflictInfo.c_str())); return TRUE; } diff --git a/PowerEditor/src/WinControls/Preference/preferenceDlg.cpp b/PowerEditor/src/WinControls/Preference/preferenceDlg.cpp index 55cc3cef..6723e676 100644 --- a/PowerEditor/src/WinControls/Preference/preferenceDlg.cpp +++ b/PowerEditor/src/WinControls/Preference/preferenceDlg.cpp @@ -233,7 +233,7 @@ void PreferenceDlg::makeCategoryList() { for (size_t i = 0, len = _wVector.size(); i < len; ++i) { - ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_ADDSTRING, 0, (LPARAM)_wVector[i]._name.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_ADDSTRING, 0, reinterpret_cast(_wVector[i]._name.c_str())); } setListSelection(0); } @@ -242,8 +242,8 @@ void PreferenceDlg::setListSelection(size_t currentSel) const { // Stupid LB API doesn't allow LB_SETSEL to be used on single select listbox, so we do it in a hard way TCHAR selStr[256]; - ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_GETTEXT, currentSel, (LPARAM)selStr); - ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_SELECTSTRING, currentSel, (LPARAM)selStr); + ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_GETTEXT, currentSel, reinterpret_cast(selStr)); + ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_SELECTSTRING, currentSel, reinterpret_cast(selStr)); } bool PreferenceDlg::renameDialogTitle(const TCHAR *internalName, const TCHAR *newName) @@ -267,14 +267,14 @@ bool PreferenceDlg::renameDialogTitle(const TCHAR *internalName, const TCHAR *ne if (txtLen >= lenMax) return false; - ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_GETTEXT, i, (LPARAM)oldName); + ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_GETTEXT, i, reinterpret_cast(oldName)); // Same name, no need to change, but operation is considered success if (lstrcmp(newName, oldName) == 0) return true; ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_DELETESTRING, i, 0); - ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_INSERTSTRING, i, (LPARAM)newName); + ::SendDlgItemMessage(_hSelf, IDC_LIST_DLGTITLE, LB_INSERTSTRING, i, reinterpret_cast(newName)); return true; } @@ -361,7 +361,7 @@ INT_PTR CALLBACK BarsDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM) for (size_t i = 0, len = localizationSwitcher.size(); i < len ; ++i) { pair localizationInfo = localizationSwitcher.getElementFromIndex(i); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_LOCALIZATION, CB_ADDSTRING, 0, (LPARAM)localizationInfo.first.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_LOCALIZATION, CB_ADDSTRING, 0, reinterpret_cast(localizationInfo.first.c_str())); } wstring lang = TEXT("English"); // Set default language as Englishs if (pNppParam->getNativeLangA()) // if nativeLangA is not NULL, then we can be sure the default language (English) is not used @@ -371,11 +371,11 @@ INT_PTR CALLBACK BarsDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM) fnW.assign(fn.begin(), fn.end()); lang = localizationSwitcher.getLangFromXmlFileName(fnW.c_str()); } - auto index = ::SendDlgItemMessage(_hSelf, IDC_COMBO_LOCALIZATION, CB_FINDSTRINGEXACT, (WPARAM)-1, (LPARAM)lang.c_str()); + auto index = ::SendDlgItemMessage(_hSelf, IDC_COMBO_LOCALIZATION, CB_FINDSTRINGEXACT, static_cast(-1), reinterpret_cast(lang.c_str())); if (index != CB_ERR) ::SendDlgItemMessage(_hSelf, IDC_COMBO_LOCALIZATION, CB_SETCURSEL, index, 0); - ETDTProc enableDlgTheme = (ETDTProc)pNppParam->getEnableThemeDlgTexture(); + ETDTProc enableDlgTheme = reinterpret_cast(pNppParam->getEnableThemeDlgTexture()); if (enableDlgTheme) enableDlgTheme(_hSelf, ETDT_ENABLETAB); @@ -882,8 +882,8 @@ INT_PTR CALLBACK SettingsDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM) ::SendDlgItemMessage(_hSelf, IDC_CHECK_CLICKABLELINK_NOUNDERLINE, BM_SETCHECK, dontUnderline, 0); ::EnableWindow(::GetDlgItem(_hSelf, IDC_CHECK_CLICKABLELINK_NOUNDERLINE), dontUnderlineState); - ::SendDlgItemMessage(_hSelf, IDC_EDIT_SESSIONFILEEXT, WM_SETTEXT, 0, (LPARAM)nppGUI._definedSessionExt.c_str()); - ::SendDlgItemMessage(_hSelf, IDC_EDIT_WORKSPACEFILEEXT, WM_SETTEXT, 0, (LPARAM)nppGUI._definedWorkspaceExt.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_EDIT_SESSIONFILEEXT, WM_SETTEXT, 0, reinterpret_cast(nppGUI._definedSessionExt.c_str())); + ::SendDlgItemMessage(_hSelf, IDC_EDIT_WORKSPACEFILEEXT, WM_SETTEXT, 0, reinterpret_cast(nppGUI._definedWorkspaceExt.c_str())); ::SendDlgItemMessage(_hSelf, IDC_CHECK_ENABLEDOCSWITCHER, BM_SETCHECK, nppGUI._doTaskList, 0); ::SendDlgItemMessage(_hSelf, IDC_CHECK_MAINTAININDENT, BM_SETCHECK, nppGUI._maitainIndent, 0); @@ -1173,7 +1173,7 @@ INT_PTR CALLBACK DefaultNewDocDlg::run_dlgProc(UINT Message, WPARAM wParam, LPAR { cmdID += IDM_FORMAT_ENCODE; getNameStrFromCmd(cmdID, str); - int index = static_cast(::SendDlgItemMessage(_hSelf, IDC_COMBO_OTHERCP, CB_ADDSTRING, 0, (LPARAM)str.c_str())); + int index = static_cast(::SendDlgItemMessage(_hSelf, IDC_COMBO_OTHERCP, CB_ADDSTRING, 0, reinterpret_cast(str.c_str()))); if (ndds._codepage == encodings[i]) selIndex = index; ::SendDlgItemMessage(_hSelf, IDC_COMBO_OTHERCP, CB_SETITEMDATA, index, (LPARAM)encodings[i]); @@ -1198,16 +1198,16 @@ INT_PTR CALLBACK DefaultNewDocDlg::run_dlgProc(UINT Message, WPARAM wParam, LPAR for (int i = L_TEXT ; i < pNppParam->L_END ; ++i) { str.clear(); - if ((LangType)i != L_USER) + if (static_cast(i) != L_USER) { - int cmdID = pNppParam->langTypeToCommandID((LangType)i); + int cmdID = pNppParam->langTypeToCommandID(static_cast(i)); if ((cmdID != -1)) { getNameStrFromCmd(cmdID, str); if (str.length() > 0) { - _langList.push_back(LangID_Name((LangType)i, str)); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_DEFAULTLANG, CB_ADDSTRING, 0, (LPARAM)str.c_str()); + _langList.push_back(LangID_Name(static_cast(i), str)); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_DEFAULTLANG, CB_ADDSTRING, 0, reinterpret_cast(str.c_str())); if (ndds._lang == i) index = _langList.size() - 1; } @@ -1545,7 +1545,7 @@ INT_PTR CALLBACK RecentFilesHistoryDlg::run_dlgProc(UINT Message, WPARAM wParam, INT_PTR CALLBACK LangMenuDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM lParam) { NppParameters *pNppParam = NppParameters::getInstance(); - NppGUI & nppGUI = (NppGUI & )pNppParam->getNppGUI(); + NppGUI & nppGUI = const_cast(pNppParam->getNppGUI()); switch (Message) { @@ -1554,22 +1554,22 @@ INT_PTR CALLBACK LangMenuDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM lP int nbLang = pNppParam->getNbLang(); for (int i = 0 ; i < nbLang ; ++i) { - ::SendDlgItemMessage(_hSelf, IDC_LIST_TABSETTNG, LB_ADDSTRING, 0, (LPARAM)pNppParam->getLangFromIndex(i)->_langName.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_LIST_TABSETTNG, LB_ADDSTRING, 0, reinterpret_cast(pNppParam->getLangFromIndex(i)->_langName.c_str())); } for (int i = L_TEXT ; i < pNppParam->L_END ; ++i) { generic_string str; - if ((LangType)i != L_USER) + if (static_cast(i) != L_USER) { - int cmdID = pNppParam->langTypeToCommandID((LangType)i); + int cmdID = pNppParam->langTypeToCommandID(static_cast(i)); if ((cmdID != -1)) { getNameStrFromCmd(cmdID, str); if (str.length() > 0) { - _langList.push_back(LangMenuItem((LangType)i, cmdID, str)); - ::SendDlgItemMessage(_hSelf, IDC_LIST_ENABLEDLANG, LB_ADDSTRING, 0, (LPARAM)str.c_str()); + _langList.push_back(LangMenuItem(static_cast(i), cmdID, str)); + ::SendDlgItemMessage(_hSelf, IDC_LIST_ENABLEDLANG, LB_ADDSTRING, 0, reinterpret_cast(str.c_str())); } } } @@ -1577,14 +1577,14 @@ INT_PTR CALLBACK LangMenuDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM lP for (size_t i = 0, len = nppGUI._excludedLangList.size(); i < len ; ++i) { - ::SendDlgItemMessage(_hSelf, IDC_LIST_DISABLEDLANG, LB_ADDSTRING, 0, (LPARAM)nppGUI._excludedLangList[i]._langName.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_LIST_DISABLEDLANG, LB_ADDSTRING, 0, reinterpret_cast(nppGUI._excludedLangList[i]._langName.c_str())); } ::SendDlgItemMessage(_hSelf, IDC_CHECK_LANGMENUCOMPACT, BM_SETCHECK, nppGUI._isLangMenuCompact?BST_CHECKED:BST_UNCHECKED, 0); ::EnableWindow(::GetDlgItem(_hSelf, IDC_BUTTON_REMOVE), FALSE); ::EnableWindow(::GetDlgItem(_hSelf, IDC_BUTTON_RESTORE), FALSE); - ETDTProc enableDlgTheme = (ETDTProc)pNppParam->getEnableThemeDlgTexture(); + ETDTProc enableDlgTheme = reinterpret_cast(pNppParam->getEnableThemeDlgTexture()); if (enableDlgTheme) enableDlgTheme(_hSelf, ETDT_ENABLETAB); @@ -1748,10 +1748,10 @@ INT_PTR CALLBACK TabSettings::run_dlgProc(UINT Message, WPARAM wParam, LPARAM/* ::SendDlgItemMessage(_hSelf, IDC_CHECK_REPLACEBYSPACE, BM_SETCHECK, nppGUI._tabReplacedBySpace, 0); int nbLang = pNppParam->getNbLang(); - ::SendDlgItemMessage(_hSelf, IDC_LIST_TABSETTNG, LB_ADDSTRING, 0, (LPARAM)TEXT("[Default]")); + ::SendDlgItemMessage(_hSelf, IDC_LIST_TABSETTNG, LB_ADDSTRING, 0, reinterpret_cast(TEXT("[Default]"))); for (int i = 0 ; i < nbLang ; ++i) { - ::SendDlgItemMessage(_hSelf, IDC_LIST_TABSETTNG, LB_ADDSTRING, 0, (LPARAM)pNppParam->getLangFromIndex(i)->_langName.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_LIST_TABSETTNG, LB_ADDSTRING, 0, reinterpret_cast(pNppParam->getLangFromIndex(i)->_langName.c_str())); } const int index2Begin = 0; ::SendDlgItemMessage(_hSelf, IDC_LIST_TABSETTNG, LB_SETCURSEL, 0, index2Begin); @@ -1760,7 +1760,7 @@ INT_PTR CALLBACK TabSettings::run_dlgProc(UINT Message, WPARAM wParam, LPARAM/* ::EnableWindow(::GetDlgItem(_hSelf, IDC_TABSIZEVAL_DISABLE_STATIC), FALSE); ::ShowWindow(::GetDlgItem(_hSelf, IDC_TABSIZEVAL_DISABLE_STATIC), SW_HIDE); - ETDTProc enableDlgTheme = (ETDTProc)pNppParam->getEnableThemeDlgTexture(); + ETDTProc enableDlgTheme = reinterpret_cast(pNppParam->getEnableThemeDlgTexture()); if (enableDlgTheme) enableDlgTheme(_hSelf, ETDT_ENABLETAB); @@ -1994,32 +1994,32 @@ INT_PTR CALLBACK PrintSettingsDlg::run_dlgProc(UINT Message, WPARAM wParam, LPAR for(size_t i = 6 ; i < 15 ; ++i) { wsprintf(intStr, TEXT("%d"), i); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTSIZE, CB_ADDSTRING, 0, (LPARAM)intStr); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTSIZE, CB_ADDSTRING, 0, (LPARAM)intStr); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTSIZE, CB_ADDSTRING, 0, reinterpret_cast(intStr)); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTSIZE, CB_ADDSTRING, 0, reinterpret_cast(intStr)); } const std::vector & fontlist = pNppParam->getFontList(); for (size_t i = 0, len = fontlist.size() ; i < len ; ++i) { - auto j = ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTNAME, CB_ADDSTRING, 0, (LPARAM)fontlist[i].c_str()); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTNAME, CB_ADDSTRING, 0, (LPARAM)fontlist[i].c_str()); + auto j = ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTNAME, CB_ADDSTRING, 0, reinterpret_cast(fontlist[i].c_str())); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTNAME, CB_ADDSTRING, 0, reinterpret_cast(fontlist[i].c_str())); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTNAME, CB_SETITEMDATA, j, (LPARAM)fontlist[i].c_str()); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTNAME, CB_SETITEMDATA, j, (LPARAM)fontlist[i].c_str()); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTNAME, CB_SETITEMDATA, j, reinterpret_cast(fontlist[i].c_str())); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTNAME, CB_SETITEMDATA, j, reinterpret_cast(fontlist[i].c_str())); } - auto index = ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTNAME, CB_FINDSTRINGEXACT, (WPARAM)-1, (LPARAM)nppGUI._printSettings._headerFontName.c_str()); + auto index = ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTNAME, CB_FINDSTRINGEXACT, static_cast(-1), reinterpret_cast(nppGUI._printSettings._headerFontName.c_str())); if (index == CB_ERR) index = 0; ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTNAME, CB_SETCURSEL, index, 0); - index = ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTNAME, CB_FINDSTRINGEXACT, (WPARAM)-1, (LPARAM)nppGUI._printSettings._footerFontName.c_str()); + index = ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTNAME, CB_FINDSTRINGEXACT, static_cast(-1), reinterpret_cast(nppGUI._printSettings._footerFontName.c_str())); if (index == CB_ERR) index = 0; ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTNAME, CB_SETCURSEL, index, 0); wsprintf(intStr, TEXT("%d"), nppGUI._printSettings._headerFontSize); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTSIZE, CB_SELECTSTRING, (WPARAM)-1, (LPARAM)intStr); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_HFONTSIZE, CB_SELECTSTRING, static_cast(-1), reinterpret_cast(intStr)); wsprintf(intStr, TEXT("%d"), nppGUI._printSettings._footerFontSize); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTSIZE, CB_SELECTSTRING, (WPARAM)-1, (LPARAM)intStr); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_FFONTSIZE, CB_SELECTSTRING, static_cast(-1), reinterpret_cast(intStr)); ::SendDlgItemMessage(_hSelf, IDC_CHECK_HBOLD, BM_SETCHECK, nppGUI._printSettings._headerFontStyle & (FONTSTYLE_BOLD ? TRUE : FALSE), 0); ::SendDlgItemMessage(_hSelf, IDC_CHECK_HITALIC, BM_SETCHECK, nppGUI._printSettings._headerFontStyle & (FONTSTYLE_ITALIC ? TRUE : FALSE), 0); @@ -2036,8 +2036,8 @@ INT_PTR CALLBACK PrintSettingsDlg::run_dlgProc(UINT Message, WPARAM wParam, LPAR for (size_t i = 0, len = varList.size() ; i < len ; ++i) { - auto j = ::SendDlgItemMessage(_hSelf, IDC_COMBO_VARLIST, CB_ADDSTRING, 0, (LPARAM)varList[i]._varDesc.c_str()); - ::SendDlgItemMessage(_hSelf, IDC_COMBO_VARLIST, CB_SETITEMDATA, j, (LPARAM)varList[i]._var.c_str()); + auto j = ::SendDlgItemMessage(_hSelf, IDC_COMBO_VARLIST, CB_ADDSTRING, 0, reinterpret_cast(varList[i]._varDesc.c_str())); + ::SendDlgItemMessage(_hSelf, IDC_COMBO_VARLIST, CB_SETITEMDATA, j, reinterpret_cast(varList[i]._var.c_str())); } ::SendDlgItemMessage(_hSelf, IDC_COMBO_VARLIST, CB_SETCURSEL, 0, 0); @@ -2134,7 +2134,7 @@ INT_PTR CALLBACK PrintSettingsDlg::run_dlgProc(UINT Message, WPARAM wParam, LPAR default : return TRUE; } ::GetDlgItemText(_hSelf, _focusedEditCtrl, str, stringSize); - ::SendDlgItemMessage(_hSelf, IDC_VIEWPANEL_STATIC, WM_SETTEXT, 0, (LPARAM)str); + ::SendDlgItemMessage(_hSelf, IDC_VIEWPANEL_STATIC, WM_SETTEXT, 0, reinterpret_cast(str)); ::GetDlgItemText(_hSelf, groupStatic, str, stringSize); generic_string title = str; title += TEXT(" "); @@ -2142,7 +2142,7 @@ INT_PTR CALLBACK PrintSettingsDlg::run_dlgProc(UINT Message, WPARAM wParam, LPAR title += str; title += TEXT(" : "); - ::SendDlgItemMessage(_hSelf, IDC_WHICHPART_STATIC, WM_SETTEXT, 0, (LPARAM)title.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_WHICHPART_STATIC, WM_SETTEXT, 0, reinterpret_cast(title.c_str())); return TRUE; } else if (HIWORD(wParam) == CBN_SELCHANGE) @@ -2288,7 +2288,7 @@ INT_PTR CALLBACK BackupDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM) if (nppGUI._useDir) ::SendDlgItemMessage(_hSelf, IDC_BACKUPDIR_CHECK, BM_SETCHECK, BST_CHECKED, 0); - ::SendDlgItemMessage(_hSelf, IDC_BACKUPDIR_EDIT, WM_SETTEXT, 0, (LPARAM)nppGUI._backupDir.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_BACKUPDIR_EDIT, WM_SETTEXT, 0, reinterpret_cast((nppGUI._backupDir.c_str()))); updateBackupGUI(); return TRUE; @@ -2302,7 +2302,7 @@ INT_PTR CALLBACK BackupDlg::run_dlgProc(UINT Message, WPARAM wParam, LPARAM) case IDC_BACKUPDIR_EDIT: { TCHAR inputDir[MAX_PATH]; - ::SendDlgItemMessage(_hSelf, IDC_BACKUPDIR_EDIT, WM_GETTEXT, MAX_PATH, (LPARAM)inputDir); + ::SendDlgItemMessage(_hSelf, IDC_BACKUPDIR_EDIT, WM_GETTEXT, MAX_PATH, reinterpret_cast(inputDir)); nppGUI._backupDir = inputDir; return TRUE; } @@ -2932,7 +2932,7 @@ INT_PTR CALLBACK SettingsOnCloudDlg::run_dlgProc(UINT Message, WPARAM wParam, LP ::SendDlgItemMessage(_hSelf, IDC_NOCLOUD_RADIO, BM_SETCHECK, !withCloud ? BST_CHECKED : BST_UNCHECKED, 0); ::SendDlgItemMessage(_hSelf, IDC_WITHCLOUD_RADIO, BM_SETCHECK, withCloud ? BST_CHECKED : BST_UNCHECKED, 0); - ::SendDlgItemMessage(_hSelf, IDC_CLOUDPATH_EDIT, WM_SETTEXT, 0, (LPARAM)nppGUI._cloudPath.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_CLOUDPATH_EDIT, WM_SETTEXT, 0, reinterpret_cast(nppGUI._cloudPath.c_str())); ::EnableWindow(::GetDlgItem(_hSelf, IDC_CLOUDPATH_EDIT), withCloud); ::EnableWindow(::GetDlgItem(_hSelf, IDD_CLOUDPATH_BROWSE_BUTTON), withCloud); } @@ -2950,7 +2950,7 @@ INT_PTR CALLBACK SettingsOnCloudDlg::run_dlgProc(UINT Message, WPARAM wParam, LP generic_string message = nppParams->isCloudPathChanged() ? TEXT("Please restart Notepad++ to take effect.") : TEXT(""); ::SetDlgItemText(_hSelf, IDC_SETTINGSONCLOUD_WARNING_STATIC, message.c_str()); - ::SendDlgItemMessage(_hSelf, IDC_CLOUDPATH_EDIT, WM_SETTEXT, 0, (LPARAM)nppGUI._cloudPath.c_str()); + ::SendDlgItemMessage(_hSelf, IDC_CLOUDPATH_EDIT, WM_SETTEXT, 0, reinterpret_cast(nppGUI._cloudPath.c_str())); ::EnableWindow(::GetDlgItem(_hSelf, IDC_CLOUDPATH_EDIT), false); ::EnableWindow(::GetDlgItem(_hSelf, IDD_CLOUDPATH_BROWSE_BUTTON), false); } diff --git a/PowerEditor/src/WinControls/StatusBar/StatusBar.cpp b/PowerEditor/src/WinControls/StatusBar/StatusBar.cpp index 9b978bb0..6c9be9cb 100644 --- a/PowerEditor/src/WinControls/StatusBar/StatusBar.cpp +++ b/PowerEditor/src/WinControls/StatusBar/StatusBar.cpp @@ -135,7 +135,7 @@ void StatusBar::adjustParts(int clientWidth) } // Tell the status bar to create the window parts. - ::SendMessage(_hSelf, SB_SETPARTS, (WPARAM)_partWidthArray.size(), (LPARAM)_lpParts); + ::SendMessage(_hSelf, SB_SETPARTS, _partWidthArray.size(), reinterpret_cast(_lpParts)); } @@ -148,7 +148,7 @@ bool StatusBar::setText(const TCHAR* str, int whichPart) else _lastSetText.clear(); - return (TRUE == ::SendMessage(_hSelf, SB_SETTEXT, whichPart, (LPARAM)_lastSetText.c_str())); + return (TRUE == ::SendMessage(_hSelf, SB_SETTEXT, whichPart, reinterpret_cast(_lastSetText.c_str()))); } assert(false and "invalid status bar index"); return false; @@ -162,5 +162,5 @@ bool StatusBar::setOwnerDrawText(const TCHAR* str) else _lastSetText.clear(); - return (::SendMessage(_hSelf, SB_SETTEXT, SBT_OWNERDRAW, (LPARAM)_lastSetText.c_str()) == TRUE); + return (::SendMessage(_hSelf, SB_SETTEXT, SBT_OWNERDRAW, reinterpret_cast(_lastSetText.c_str())) == TRUE); } diff --git a/PowerEditor/src/WinControls/VerticalFileSwitcher/VerticalFileSwitcherListView.cpp b/PowerEditor/src/WinControls/VerticalFileSwitcher/VerticalFileSwitcherListView.cpp index e63100e8..f211ee99 100644 --- a/PowerEditor/src/WinControls/VerticalFileSwitcher/VerticalFileSwitcherListView.cpp +++ b/PowerEditor/src/WinControls/VerticalFileSwitcher/VerticalFileSwitcherListView.cpp @@ -146,7 +146,7 @@ void VerticalFileSwitcherListView::initList() ListView_InsertItem(_hSelf, &item); if (isExtColumn) { - ListView_SetItemText(_hSelf, i, 1, (LPTSTR)::PathFindExtension(fileNameStatus._fn.c_str())); + ListView_SetItemText(_hSelf, i, 1, ::PathFindExtension(fileNameStatus._fn.c_str())); } } ListView_SetItemState(_hSelf, taskListInfo._currentIndex, LVIS_SELECTED | LVIS_FOCUSED, LVIS_SELECTED | LVIS_FOCUSED); diff --git a/PowerEditor/src/WinControls/shortcut/shortcut.cpp b/PowerEditor/src/WinControls/shortcut/shortcut.cpp index 422c6fec..3a0ecb9d 100644 --- a/PowerEditor/src/WinControls/shortcut/shortcut.cpp +++ b/PowerEditor/src/WinControls/shortcut/shortcut.cpp @@ -812,7 +812,7 @@ void ScintillaKeyMap::showCurrentSettings() { } void ScintillaKeyMap::updateListItem(int index) { - ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_INSERTSTRING, index, (LPARAM)toString(index).c_str()); + ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_INSERTSTRING, index, reinterpret_cast(toString(index).c_str())); ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_DELETESTRING, index+1, 0); } @@ -828,12 +828,12 @@ INT_PTR CALLBACK ScintillaKeyMap::run_dlgProc(UINT Message, WPARAM wParam, LPARA for (size_t i = 0 ; i < nrKeys ; ++i) { - ::SendDlgItemMessage(_hSelf, IDC_KEY_COMBO, CB_ADDSTRING, 0, (LPARAM)namedKeyArray[i].name); + ::SendDlgItemMessage(_hSelf, IDC_KEY_COMBO, CB_ADDSTRING, 0, reinterpret_cast(namedKeyArray[i].name)); } for(size_t i = 0; i < _size; ++i) { - ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_ADDSTRING, 0, (LPARAM)toString(i).c_str()); + ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_ADDSTRING, 0, reinterpret_cast(toString(i).c_str())); } ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_SETCURSEL, 0, 0); @@ -885,7 +885,7 @@ INT_PTR CALLBACK ScintillaKeyMap::run_dlgProc(UINT Message, WPARAM wParam, LPARA { if (res == static_cast(oldsize)) { - ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_INSERTSTRING, (WPARAM)-1, (LPARAM)toString(res).c_str()); + ::SendDlgItemMessage(_hSelf, IDC_LIST_KEYS, LB_INSERTSTRING, static_cast(-1), reinterpret_cast(toString(res).c_str())); } else { //update current generic_string, can happen if it was disabled