From aa8f4373e0c9614b7d64a5760c0fa071b1c1edc9 Mon Sep 17 00:00:00 2001 From: George Hazan Date: Sat, 21 Jul 2012 11:11:20 +0000 Subject: - MZeroedObject used instead of the operator new() inlining - SAFE_DELETE of local objects removed from ICQ git-svn-id: http://svn.miranda-ng.org/main/trunk@1092 1316c22d-e87f-b044-9b9b-93d7a3e3ba9c --- plugins/ModernOpt/src/modernopt.cpp | 31 ++++++++++++------------------- 1 file changed, 12 insertions(+), 19 deletions(-) (limited to 'plugins/ModernOpt/src') diff --git a/plugins/ModernOpt/src/modernopt.cpp b/plugins/ModernOpt/src/modernopt.cpp index 1345631268..16ed0738ff 100644 --- a/plugins/ModernOpt/src/modernopt.cpp +++ b/plugins/ModernOpt/src/modernopt.cpp @@ -52,15 +52,8 @@ static int g_iSectionRestore = 0; static int ModernOptionsObject_Comparator(const ModernOptionsObject *ptr1, const ModernOptionsObject *ptr2); -struct ModernOptionsData +struct ModernOptionsData : public MZeroedObject { - __inline void* operator new( size_t size ) - { return calloc( 1, size ); - } - __inline void operator delete( void* p ) - { free( p ); - } - ModernOptionsData(): pObjectList(1, ModernOptionsObject_Comparator) {} LIST pObjectList; @@ -149,9 +142,9 @@ static int ModernOptionsObject_Comparator(const ModernOptionsObject *ptr1, const } void li_List_Destruct(LIST &pList, ItemDestuctor pItemDestructor) -{ +{ int i=0; - for (i=0; i= dat->pObjectList.getCount())) + if ((iPage < 0) || (iPage >= dat->pObjectList.getCount())) return NULL; dat->iPage = iPage; @@ -389,7 +382,7 @@ static HWND ModernOptUI_ShowPage_Impl(HWND hwndDlg, struct ModernOptionsData *da if (ignoreObj->optObject.iType == MODERNOPT_TYPE_IGNOREOBJECT) ModernOptIgnore_AddItem(&ignoreObj->optObject); } } } - } + } else ShowWindow(obj->hwnd, SW_SHOW); ShowWindow(GetDlgItem(hwndDlg, IDC_BTN_EXPERT), (obj->optObject.lpzClassicGroup || obj->optObject.lpzClassicPage) ? SW_SHOW : SW_HIDE); @@ -422,7 +415,7 @@ static void ModernOptUI_ShowPage(HWND hwndDlg, struct ModernOptionsData *dat, in RECT rcWnd; GetWindowRect(dat->pObjectList[i]->hwnd, &rcWnd); dx += rcWnd.bottom - rcWnd.top;// + 30; } - } + } else if (dat->pObjectList[i]->hwnd) { ShowWindow(dat->pObjectList[i]->hwnd, SW_HIDE); @@ -436,7 +429,7 @@ static void ModernOptUI_SelectSection(HWND hwndDlg, struct ModernOptionsData *da HWND hwndTree = GetDlgItem(hwndDlg, IDC_TV_SUBSECTIONS); dat->iSection = iSection; - + SendMessage(hwndTree, WM_SETREDRAW, FALSE, 0); TreeView_DeleteAllItems(hwndTree); for (i = 0; i < dat->pObjectList.getCount(); ++i) { @@ -449,8 +442,8 @@ static void ModernOptUI_SelectSection(HWND hwndDlg, struct ModernOptionsData *da if (obj->optObject.iType == MODERNOPT_TYPE_SECTIONPAGE) { ModernOptUI_ShowPage(hwndDlg, dat, i); break; - } - + } + if (obj->optObject.iType == MODERNOPT_TYPE_SUBSECTIONPAGE) { TVINSERTSTRUCT tvis = {0}; tvis.hParent = TVI_ROOT; @@ -472,7 +465,7 @@ static void ModernOptUI_SelectSection(HWND hwndDlg, struct ModernOptionsData *da ShowWindow(hwndTree, SW_SHOW); RedrawWindow(hwndTree, NULL, NULL, RDW_INVALIDATE); TreeView_Select(hwndTree, TreeView_GetRoot(hwndTree), TVGN_CARET); - } + } else ShowWindow(hwndTree, SW_HIDE); } @@ -486,7 +479,7 @@ static INT_PTR svcModernOpt_Impl(WPARAM wParam, LPARAM lParam) NotifyEventHooks(hevtModernOpt_Initialize, (WPARAM)dat, 0); hwndModernOpt = CreateDialogParam(hInst, MAKEINTRESOURCE(IDD_MODERNOPT), NULL, ModernOptDlgProc, (LPARAM)dat); ShowWindow(hwndModernOpt, SW_SHOW); - } + } else SetForegroundWindow(hwndModernOpt); return 0; -- cgit v1.2.3