tyler.durden/notepad-plus-plus
1
0
Fork 0
mirror of https://github.com/notepad-plus-plus/notepad-plus-plus.git synced 2025-07-26 23:34:44 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix Incomplete styler.xml crashes Notepad++ issue

Browse Source 
And do some refactoring.

Fix #12101
This commit is contained in:
Don Ho 2023-11-06 00:40:38 +01:00
parent 950236bb8f
commit dc6bfedb90
3 changed files with 110 additions and 90 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
PowerEditor/src/Parameters.h
View File

@ -443,7 +443,8 @@ struct StyleArray
void clear() { _styleVect.clear(); }; void clear() { _styleVect.clear(); };
Style& getStyler(size_t index) { Style& getStyler(size_t index) {
assert(index < _styleVect.size()); if (index >= _styleVect.size())
throw std::out_of_range("Styler index out of range");
return _styleVect[index]; return _styleVect[index];
}; };

98
PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.cpp
View File

@ -1162,6 +1162,42 @@ void WordStyleDlg::create(int dialogID, bool isRTL, bool msgDestParent)
} }
} }
void WordStyleDlg::doDialog(bool isRTL)
{
if (!isCreated())
{
create(IDD_STYLER_DLG, isRTL);
prepare2Cancel();
}
if (!::IsWindowVisible(_hSelf))
{
prepare2Cancel();
}
display();
}
void WordStyleDlg::prepare2Cancel()
{
_styles2restored = (NppParameters::getInstance()).getLStylerArray();
_gstyles2restored = (NppParameters::getInstance()).getGlobalStylers();
_gOverride2restored = (NppParameters::getInstance()).getGlobalOverrideStyle();
}
void WordStyleDlg::redraw(bool forceUpdate) const
{
_pFgColour->redraw(forceUpdate);
_pBgColour->redraw(forceUpdate);
::InvalidateRect(_hStyleInfoStaticText, NULL, TRUE);
::UpdateWindow(_hStyleInfoStaticText);
}
void WordStyleDlg::restoreGlobalOverrideValues()
{
GlobalOverride& gOverride = (NppParameters::getInstance()).getGlobalOverrideStyle();
gOverride = _gOverride2restored;
}
void WordStyleDlg::apply() void WordStyleDlg::apply()
{ {
@ -1175,3 +1211,65 @@ void WordStyleDlg::apply()
::SendMessage(_hParent, WM_UPDATESCINTILLAS, 0, 0); ::SendMessage(_hParent, WM_UPDATESCINTILLAS, 0, 0);
::SendMessage(_hParent, WM_UPDATEMAINMENUBITMAPS, 0, 0); ::SendMessage(_hParent, WM_UPDATEMAINMENUBITMAPS, 0, 0);
} }
void WordStyleDlg::addLastThemeEntry()
{
NppParameters& nppParamInst = NppParameters::getInstance();
ThemeSwitcher& themeSwitcher = nppParamInst.getThemeSwitcher();
std::pair<generic_string, generic_string>& themeInfo = themeSwitcher.getElementFromIndex(themeSwitcher.size() - 1);
::SendMessage(_hSwitch2ThemeCombo, CB_ADDSTRING, 0, reinterpret_cast<LPARAM>(themeInfo.first.c_str()));
}
Style& WordStyleDlg::getCurrentStyler()
{
int32_t styleIndex = static_cast<int32_t>(::SendDlgItemMessage(_hSelf, IDC_STYLES_LIST, LB_GETCURSEL, 0, 0));
if (styleIndex == LB_ERR)
styleIndex = 0;
try {
if (_currentLexerIndex == 0)
{
return _globalStyles.getStyler(styleIndex);
}
else
{
LexerStyler& lexerStyler = _lsArray.getLexerFromIndex(_currentLexerIndex - 1);
return lexerStyler.getStyler(styleIndex);
}
}
catch (...)
{
return _globalStyles.getStyler(0);
}
}
void WordStyleDlg::enableFontStyle(bool isEnable)
{
::EnableWindow(_hCheckBold, isEnable);
::EnableWindow(_hCheckItalic, isEnable);
::EnableWindow(_hCheckUnderline, isEnable);
}
long WordStyleDlg::notifyDataModified()
{
_isDirty = true;
_isThemeDirty = true;
::EnableWindow(::GetDlgItem(_hSelf, IDC_SAVECLOSE_BUTTON), TRUE);
return TRUE;
}
void WordStyleDlg::showGlobalOverrideCtrls(bool show)
{
if (show)
{
updateGlobalOverrideCtrls();
}
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_FG_CHECK), show ? SW_SHOW : SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_BG_CHECK), show ? SW_SHOW : SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_FONT_CHECK), show ? SW_SHOW : SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_FONTSIZE_CHECK), show ? SW_SHOW : SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_BOLD_CHECK), show ? SW_SHOW : SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_ITALIC_CHECK), show ? SW_SHOW : SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_UNDERLINE_CHECK), show ? SW_SHOW : SW_HIDE);
_isShownGOCtrls = show;
}

99
PowerEditor/src/WinControls/ColourPicker/WordStyleDlg.h
View File

@ -62,50 +62,13 @@ public :
WordStyleDlg() = default; WordStyleDlg() = default;
void create(int dialogID, bool isRTL = false, bool msgDestParent = true) override; void create(int dialogID, bool isRTL = false, bool msgDestParent = true) override;
void doDialog(bool isRTL = false);
void doDialog(bool isRTL = false) { void prepare2Cancel();
if (!isCreated()) void redraw(bool forceUpdate = false) const override;
{ void restoreGlobalOverrideValues();
create(IDD_STYLER_DLG, isRTL);
prepare2Cancel();
}
if (!::IsWindowVisible(_hSelf))
{
prepare2Cancel();
}
display();
};
void prepare2Cancel() {
_styles2restored = (NppParameters::getInstance()).getLStylerArray();
_gstyles2restored = (NppParameters::getInstance()).getGlobalStylers();
_gOverride2restored = (NppParameters::getInstance()).getGlobalOverrideStyle();
};
void redraw(bool forceUpdate = false) const override {
_pFgColour->redraw(forceUpdate);
_pBgColour->redraw(forceUpdate);
::InvalidateRect(_hStyleInfoStaticText, NULL, TRUE);
::UpdateWindow(_hStyleInfoStaticText);
};
void restoreGlobalOverrideValues() {
GlobalOverride & gOverride = (NppParameters::getInstance()).getGlobalOverrideStyle();
gOverride = _gOverride2restored;
};
void apply(); void apply();
void addLastThemeEntry();
void addLastThemeEntry() {
NppParameters& nppParamInst = NppParameters::getInstance();
ThemeSwitcher & themeSwitcher = nppParamInst.getThemeSwitcher();
std::pair<generic_string, generic_string> & themeInfo = themeSwitcher.getElementFromIndex(themeSwitcher.size() - 1);
::SendMessage(_hSwitch2ThemeCombo, CB_ADDSTRING, 0, reinterpret_cast<LPARAM>(themeInfo.first.c_str()));
};
bool selectThemeByName(const TCHAR* themeName); bool selectThemeByName(const TCHAR* themeName);
bool goToSection(const TCHAR* sectionNames); // sectionNames is formed as following: "Language name:Style name" bool goToSection(const TCHAR* sectionNames); // sectionNames is formed as following: "Language name:Style name"
// ex: "Global Styles:EOL custom color" will set Language on "Global Styles", then set Style on "EOL custom color" if both are found. // ex: "Global Styles:EOL custom color" will set Language on "Global Styles", then set Style on "EOL custom color" if both are found.
@ -146,25 +109,11 @@ private :
bool _isThemeDirty = false; bool _isThemeDirty = false;
bool _isShownGOCtrls = false; bool _isShownGOCtrls = false;
std::pair<intptr_t, intptr_t> goToPreferencesSettings();
intptr_t CALLBACK run_dlgProc(UINT Message, WPARAM wParam, LPARAM lParam) override; intptr_t CALLBACK run_dlgProc(UINT Message, WPARAM wParam, LPARAM lParam) override;
Style& getCurrentStyler();
Style & getCurrentStyler() {
int32_t styleIndex = static_cast<int32_t>(::SendDlgItemMessage(_hSelf, IDC_STYLES_LIST, LB_GETCURSEL, 0, 0));
if (styleIndex == LB_ERR)
styleIndex = 0;
if (_currentLexerIndex == 0)
{
return _globalStyles.getStyler(styleIndex);
}
else
{
LexerStyler & lexerStyler = _lsArray.getLexerFromIndex(_currentLexerIndex - 1);
return lexerStyler.getStyler(styleIndex);
}
};
bool getStyleName(TCHAR *styleName, const size_t styleNameLen); bool getStyleName(TCHAR *styleName, const size_t styleNameLen);
@ -179,40 +128,12 @@ private :
void updateUserKeywords(); void updateUserKeywords();
void switchToTheme(); void switchToTheme();
void updateThemeName(const generic_string& themeName); void updateThemeName(const generic_string& themeName);
std::pair<intptr_t, intptr_t> goToPreferencesSettings();
void loadLangListFromNppParam(); void loadLangListFromNppParam();
void enableFontStyle(bool isEnable);
void enableFontStyle(bool isEnable) { long notifyDataModified();
::EnableWindow(_hCheckBold, isEnable);
::EnableWindow(_hCheckItalic, isEnable);
::EnableWindow(_hCheckUnderline, isEnable);
};
long notifyDataModified() {
_isDirty = true;
_isThemeDirty = true;
::EnableWindow(::GetDlgItem(_hSelf, IDC_SAVECLOSE_BUTTON), TRUE);
return TRUE;
};
void setStyleListFromLexer(int index); void setStyleListFromLexer(int index);
void setVisualFromStyleList(); void setVisualFromStyleList();
void updateGlobalOverrideCtrls(); void updateGlobalOverrideCtrls();
void showGlobalOverrideCtrls(bool show);
void showGlobalOverrideCtrls(bool show) {
if (show)
{
updateGlobalOverrideCtrls();
}
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_FG_CHECK), show?SW_SHOW:SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_BG_CHECK), show?SW_SHOW:SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_FONT_CHECK), show?SW_SHOW:SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_FONTSIZE_CHECK), show?SW_SHOW:SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_BOLD_CHECK), show?SW_SHOW:SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_ITALIC_CHECK), show?SW_SHOW:SW_HIDE);
::ShowWindow(::GetDlgItem(_hSelf, IDC_GLOBAL_UNDERLINE_CHECK), show?SW_SHOW:SW_HIDE);
_isShownGOCtrls = show;
};
void applyCurrentSelectedThemeAndUpdateUI(); void applyCurrentSelectedThemeAndUpdateUI();
}; };