Use CheckDlgButton() instead of SetDlgItemChecked()

diff --git a/Src/PluginsListDlg.cpp b/Src/PluginsListDlg.cpp
index 1923d23..933d7df 100644 (file)
--- a/Src/PluginsListDlg.cpp
+++ b/Src/PluginsListDlg.cpp
@@ -51,7 +51,7 @@ BOOL PluginsListDlg::OnInitDialog()
        AddPlugins();
        m_list.SetItemState(0, LVIS_SELECTED, LVIS_SELECTED);
 
-       SetDlgItemChecked(IDC_PLUGINS_ENABLE, 
+       CheckDlgButton(IDC_PLUGINS_ENABLE, 
                GetOptionsMgr()->GetBool(OPT_PLUGINS_ENABLED) ? BST_CHECKED : BST_UNCHECKED);
 
        return FALSE;  // return TRUE unless you set the focus to a control

diff --git a/Src/PropColors.cpp b/Src/PropColors.cpp
index f690469..77bf75a 100644 (file)
--- a/Src/PropColors.cpp
+++ b/Src/PropColors.cpp
@@ -138,7 +138,7 @@ void PropMergeColors::BrowseColor(CColorButton & colorButton)
 
 void PropMergeColors::UpdateTextColorButtonState(int checkboxId, CColorButton &btn)
 {
-       SetDlgItemChecked(checkboxId, btn.GetColor() != COLOR_NONE);
+       CheckDlgButton(checkboxId, btn.GetColor() != COLOR_NONE ? BST_CHECKED : BST_UNCHECKED);
        btn.ShowWindow(btn.GetColor() != COLOR_NONE ? SW_SHOW : SW_HIDE);
 }
 

diff --git a/Src/TrDialogs.h b/Src/TrDialogs.h
index 921a792..0d2bd04 100644 (file)
--- a/Src/TrDialogs.h
+++ b/Src/TrDialogs.h
@@ -5,12 +5,6 @@
 namespace DlgUtils\r
 {\r
        template <class T>\r
-       void SetDlgItemChecked(T* dlg, unsigned id, unsigned value)\r
-       {\r
-               dlg->SendDlgItemMessage(id, BM_SETCHECK, value);\r
-       }\r
-\r
-       template <class T>\r
        void EnableDlgItem(T* dlg, unsigned id, bool enabled)\r
        {\r
                dlg->GetDlgItem(id)->EnableWindow(enabled);\r
@@ -44,7 +38,6 @@ public:
        CTrDialog(LPCTSTR lpszTemplateName, CWnd *pParentWnd = NULL) : CDialog(lpszTemplateName, pParentWnd) {}\r
 \r
        virtual BOOL OnInitDialog();\r
-       void SetDlgItemChecked(unsigned id, unsigned value) { DlgUtils::SetDlgItemChecked(this, id, value); }\r
        void EnableDlgItem(unsigned id, bool enabled) { DlgUtils::EnableDlgItem(this, id, enabled); }\r
        void SetDlgItemFocus(unsigned id) { DlgUtils::SetDlgItemFocus(this, id); }\r
        unsigned GetDlgItemText(unsigned id, String& text) { return DlgUtils::GetDlgItemText(this, id, text); }\r
@@ -62,7 +55,6 @@ public:
                : CPropertyPage(lpszTemplateName, nIDCaption, dwSize) {}\r
 \r
        virtual BOOL OnInitDialog();\r
-       void SetDlgItemChecked(unsigned id, bool value) { DlgUtils::SetDlgItemChecked(this, id, value); }\r
        void EnableDlgItem(unsigned id, bool enabled) { DlgUtils::EnableDlgItem(this, id, enabled); }\r
        void SetDlgItemFocus(unsigned id) { DlgUtils::SetDlgItemFocus(this, id); }\r
        unsigned GetDlgItemText(unsigned id, String& text) { return DlgUtils::GetDlgItemText(this, id, text); }\r
@@ -77,7 +69,6 @@ public:
                UINT nStyle, UINT nID);\r
        virtual BOOL Create(CWnd* pParentWnd, UINT nIDTemplate,\r
                UINT nStyle, UINT nID);\r
-       void SetDlgItemChecked(unsigned id, bool value) { DlgUtils::SetDlgItemChecked(this, id, value); }\r
        void EnableDlgItem(unsigned id, bool enabled) { DlgUtils::EnableDlgItem(this, id, enabled); }\r
        void SetDlgItemFocus(unsigned id) { DlgUtils::SetDlgItemFocus(this, id); }\r
        unsigned GetDlgItemText(unsigned id, String& text) { return DlgUtils::GetDlgItemText(this, id, text); }\r